Skip to content

Update dependencies - server-migration #92

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

Closed
wants to merge 48 commits into from

Conversation

crod951
Copy link

@crod951 crod951 commented Feb 27, 2025

  1. Updated node version to 22.14.0 (current LTS)
  2. Updated all non-breaking dependencies that were out of date
  3. Resolved any errors/warnings with updated dependencies
    • <th> tags need to be wrapped with <tr> in <thead> and <tbody>
    • The element inside <transition> is expected to have a v-if or v-show directive

NOTE: merge server-migration into proxy after this PR is merged

unenglishable and others added 30 commits December 22, 2023 11:57
… production

when app is compiled without the env variable set, should be set to the
string "$VUE_APP_BACKEND_URL/config.js"

modified the start-nginx.sh script to also run on index.html, which
should replace that variable during runtime
feat(profile): convert profile page to use new api
@crod951 crod951 requested a review from akinsey February 27, 2025 22:24
@crod951
Copy link
Author

crod951 commented Feb 27, 2025

closing for now, need to fix some things

@crod951 crod951 closed this Feb 27, 2025
@crod951 crod951 deleted the update-dependencies-migration branch February 27, 2025 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants