Skip to content

Update Dependencies #91

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 15 commits into from
Closed

Update Dependencies #91

wants to merge 15 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

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
…date changes
…s-loader to latest versions
…d dependencies
…rsions to match updates to server project
Copy link
Member

@akinsey akinsey left a comment

Choose a reason for hiding this comment

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

Is there anyway to get these version upgrades into the main branch or is that going to be a huge hassle? I think these changes probably should go into both main and proxy

@crod951
Copy link
Author

crod951 commented Feb 27, 2025

Is there anyway to get these version upgrades into the main branch or is that going to be a huge hassle? I think these changes probably should go into both main and proxy

good call, I'm gonna close this commit and make a new one to merge into server-migration then once that's merge, we can merge server-migration into proxy to get the updates

@crod951 crod951 closed this Feb 27, 2025
@crod951 crod951 deleted the update-dependencies branch February 28, 2025 00:21
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.

None yet

2 participants