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

chore: Migrate client to vite! #844

Merged
merged 18 commits into from
Nov 24, 2023
Merged

chore: Migrate client to vite! #844

merged 18 commits into from
Nov 24, 2023

Conversation

msarcev
Copy link
Collaborator

@msarcev msarcev commented Nov 22, 2023 •
edited
Loading

Description of change

Migrates the client bundler to vite

  • Test that everything is good on legary and chrysalis searches
  • Make aliases work with eslint (no errors on "~/*" imports)

Type of change

  • Enhancement (a non-breaking change which adds functionality)

msarcev added 18 commits November 22, 2023 19:03
@msarcev
feat: Migrate to vite bundler [WiP]
158f09a 
Mostly works but some issues with svg loading and such
@msarcev
chore: Update vite.config related to SVGs
b72eaee
@msarcev
chore: Remove react-scripts!!!
7445d06
@msarcev
chore: Fix vite svg imports and ReactComponent typing + Silence eslin…
16c326b 
…t about svg import not resolved
@msarcev
chore: Remove accidentally commited wasm from git (.gitignore it)
1112045
@msarcev
chore: Add Vite-Aliases! + Fix image resource relative paths in scss …
1686b0f 
…with vite-alias
@msarcev
chore: Remove unused react-scripts declaration helper
8ead65d
@msarcev
chore: Set port 3000 for vite preview
4c7644a
@msarcev
chore: Remove craco config
4cef90c
@msarcev
fix: Fix broken package.json
e0623bd
@msarcev
chore: Remove A LOT of unused npm dependencies (needs torough testing…
54ee9f5 
… of chrysalis and legacy pages)
@msarcev
chore: Reinstall @types/d3-transition to avoid eslint errors
97d731b
@msarcev
chore: Make alias import work with eslint and ts + Simplify .eslintrc…
33292fa 
… and SHRINK file by ~5700%
@msarcev
chore: Update ALL imports in src/** to use aliases
e369e1d
@msarcev
Merge branch 'dev' into chore/migrate-client-to-vite
c984b1e
@msarcev
chore: Bump client to v3.3.2
1300860
@msarcev
chore: Re-enable tests with vitest and hook into build
d34d49c
@msarcev
chore: Bump api version to 3.3.2 + Update CHANGELOGS
d46fc52
@panteleymonchuk panteleymonchuk self-requested a review November 24, 2023 11:56
panteleymonchuk
panteleymonchuk approved these changes Nov 24, 2023
View reviewed changes
Copy link
Collaborator

@panteleymonchuk panteleymonchuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@msarcev msarcev merged commit 0f30e83 into dev Nov 24, 2023
1 check passed
@msarcev msarcev deleted the chore/migrate-client-to-vite branch November 24, 2023 11:57
@msarcev msarcev restored the chore/migrate-client-to-vite branch November 24, 2023 14:58
@begonaalvarezd begonaalvarezd deleted the chore/migrate-client-to-vite branch December 12, 2023 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@panteleymonchuk panteleymonchuk panteleymonchuk approved these changes

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
@msarcev @panteleymonchuk