Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Replace CRA/WebPack with Vite #821

Merged
merged 108 commits into from
Dec 15, 2023
Merged

Replace CRA/WebPack with Vite #821

merged 108 commits into from
Dec 15, 2023

Conversation

travjenkins
Copy link
Member

@travjenkins travjenkins commented Nov 10, 2023 •
edited
Loading

Issues

#824
#875

Changes

Build

  • Replace CRA with Vite since CRA is lightly supported these days and was causing a lot of our vulnerabilities. These were not a huge issue as most were dev dependencies... but still feel better on something with active development

NPM

  • Heavily clean up package.json to remove old scripts we are not actually using

Licenses

  • Update the list of allowed licenses for licensee. This means we can not break the build commit if there is a change in the licenses.

875

Tests

Manually tested

  • Lots of hitting pages in development mode and preview mode

Screenshots

No audit vulnerabilities
image

All licenses are now handled properly
image

@travjenkins
Work to switch us over to using vite
2fff1a2
@travjenkins
Cleaning up paths
91af235
@travjenkins
Getting errors to popup in UI during dev
1bef9df
@travjenkins
Getting node polyfills pulled over
6cb19d4 
Making sure the typescript config is set
@travjenkins
Adding a quick test to make sure we have some test there
8755aa9
@travjenkins
Cleaning up old tsts
8ebdacf
@travjenkins
Getting dynamic stuff worked into the index.html file
74695e9
@travjenkins
Remove webpack stuff
7033af2
@travjenkins
Moving test to folder
5917cc4
@travjenkins
Switching config from ts to js
587a895 
Fixing lint issue
@travjenkins
Adding compression in
a659de4
@travjenkins
Removing more old testing libs until we decide to actually use them
3b620de 
Cleaning up old files
@travjenkins
adding image compression
36b92f6
@travjenkins
running audit fix
65a8e45 
Removing app rewired
@travjenkins
Removing some stuff we don't use anymore
04b5beb
@travjenkins
removing react component svg for now
296a6cb
@travjenkins
Adding port for preview so logins work
f3cd625
@travjenkins
comment
12a6ac7
@travjenkins
Removing eslint-plugin-jest-dom
24a3933
@travjenkins
Opening by default
c477493 
Removing serve
@travjenkins
Removing all reference of jest
11d8195
@travjenkins
correcting variable
6f025df
@travjenkins
Fixing name
b1f25e1
@travjenkins
close when port already opened
0b81b75
@travjenkins
Removing react svg loading as we don't need this right now
9b3a4be
@travjenkins
Removing react svg loading as we don't need this right now
f06176b
@travjenkins
Remove the webpack prefetch stuff
885e7fb
@travjenkins
Removing the lazy loading
62ec204
@travjenkins
saving replace lazy load as comments
d8f825c
@travjenkins
Only having the main pages included by default and lazy loading in cr…
22afee1 
…eate
@travjenkins
Fixing issues with audit
55d7a96
@travjenkins
Updating the building blocks section
a4a90af
@travjenkins
Renaming to make it more clear
595cce2
@travjenkins
Merge remote-tracking branch 'origin/main' into travjenkins/deps_postCSS
3bb0030
@travjenkins
Merge remote-tracking branch 'origin/main' into travjenkins/deps_postCSS
d6e8ff9
@travjenkins
Merge remote-tracking branch 'origin/main' into travjenkins/deps_postCSS
c56284f
@travjenkins
Getting monaco editor wired up and loading from our deployment
f99e10f
@travjenkins
Updating the monaco react version
c7a0727
@travjenkins
Updating grouping the vite config
46d34c1 
Adding interop for callouts vite made
@travjenkins
No errors/warnings being thrown anymore so removing
10392ff
@travjenkins
I don't think the nested arrays was needed
f8c3e4f
@travjenkins
Removing logging
7192605
@travjenkins
Adding build to preview to make sure folks use the latest
be339d9
@travjenkins
Cleaning up unused variables
323f185
@travjenkins
Adding in a version mark
9573e29
@travjenkins
Add in writing version file to meta.json
34b7801 
Switching vite config to be typescript
Updating types
@travjenkins
Ignoring vite config to make it easier to make changes
6aba55c
@travjenkins
Adding typescript to list
993b84d
@travjenkins
Merge branch 'main' into travjenkins/deps_postCSS
935046c
@travjenkins
Merge branch 'main' into travjenkins/deps_postCSS
14871cc
@travjenkins
Using a newer compression library
7f53a8d
@travjenkins
Removing how we loaded in the version to using the defined version
f0fb6a4
@travjenkins
Updating nginx to serve up the meta.json file
c23f864 
Commenting
@travjenkins
Using the version variable directly
2a56cca
@travjenkins
Adding a defined app version variable
f590b97 
Writing the version to a json file during build
Switching compression plugins
@travjenkins
Making it so we only compress the specific files we want
15c23a1
@travjenkins
Getting the settings for compression worked out
625cca4
@travjenkins
Updating read me
4e9e3a2 
Removing build from preview
@travjenkins travjenkins merged commit 5e6db7d into main Dec 15, 2023
@travjenkins travjenkins deleted the travjenkins/deps_postCSS branch December 15, 2023 00:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kiahna-tucker kiahna-tucker kiahna-tucker approved these changes

@skord skord Awaiting requested review from skord

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@travjenkins @kiahna-tucker