-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: migrate from CRA4 to Vite, Storybook to v8, jest to Vitest #7285
base: develop
Are you sure you want to change the base?
feat: migrate from CRA4 to Vite, Storybook to v8, jest to Vitest #7285
Conversation
@KenLSM see chromatic useful. will ensure the migration has 0 frontend changes!!! |
@justynoh this PR officially has no regressions hehehehe |
@justynoh added datadog chunking back to the pipeline. Can push to staging to try? |
0a96443
to
9f75cd7
Compare
2c7a9e4
to
9177241
Compare
…tial_vite_config_and_new_eslint_rules # Conflicts: # frontend/.storybook/main.js # frontend/package-lock.json # frontend/package.json # frontend/src/components/Field/Attachment/Attachment.tsx # frontend/src/components/Searchbar/Searchbar.stories.tsx # frontend/src/features/admin-form/create/builder-and-design/BuilderAndDesignDrawer/FieldListDrawer/field-panels/MyInfoPanel.tsx # frontend/src/features/admin-form/share/ShareFormModal.tsx # frontend/src/features/public-form/PublicFormContext.tsx # frontend/src/hooks/useMdComponents.tsx # playwright.config.ts # shared/constants/form.ts
…tial_vite_config_and_new_eslint_rules # Conflicts: # frontend/.storybook/main.js
f3f55e9
to
1754bee
Compare
1754bee
to
2998079
Compare
62311bd
to
e06fa47
Compare
1d546e2
to
7fba8d0
Compare
7fba8d0
to
dabcfa1
Compare
88d0aaf
to
4561bf8
Compare
@@ -8,6 +8,8 @@ | |||
"jszip": "^3.7.1", | |||
"libphonenumber-js": "^1.10.48", | |||
"lodash": "^4.17.21", | |||
"tweetnacl": "^1.0.3", | |||
"tweetnacl-util": "^0.15.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kevin9foong btw we forgot to add these two packages in to /shared
💀.
Finally solved final piece of why chromatic publishing fails when using vite/rollup. webpack
is a little looser in resolving imports thus we didn't notice it while rollup/pnpm
are much stricter.
This PR moves the frontend toolchain from CRA4 to Vite.
Generally everything seems to be working fine, including decryption workers.
Requires a bunch of changes that has to be performed all at the same time:
TODO: