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

Migrate to vite #361

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Migrate to vite #361

wants to merge 8 commits into from

Conversation

ChristophB
Copy link
Member

@ChristophB ChristophB commented Feb 21, 2025 •
edited
Loading

This PR contains a lot of changes to the code base:

  • Upgraded dependencies
  • Migrated from webpack to vite
  • Updated linting rules and fixed all resulting errors and warnings
  • There are now some npm scripts in place for linting, formatting, running the dev server, etc.

You may need to delete your .quasar folder before executing npm install.

@ChristophB ChristophB marked this pull request as ready for review February 26, 2025 14:38
@ChristophB ChristophB requested review from fmatthies and RalphBln and removed request for fmatthies February 26, 2025 14:38
@ChristophB
Migrate to vite
cb153ba 
- Refactor project files
- Reformat code
@ChristophB
Upgrade dependencies
77a4cd8
@ChristophB
Improve Home.vue
8158f86 
Apply minor code improvements and add a notification if statistics could not be loaded
@ChristophB
Simplify code of organisation pages
2f55554
@ChristophB
Remove uses of v-t
932bd75 
v-t is deprecated since vue-i18n v11
@ChristophB
Switch back to i18n JSON format
8aa51eb 
VS Code extension does not support write to TS files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RalphBln RalphBln Awaiting requested review from RalphBln

@fmatthies fmatthies Awaiting requested review from fmatthies

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

Successfully merging this pull request may close these issues.
1 participant
@ChristophB