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.

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

Upgrade Next version #70

Merged
merged 1 commit into from
Jan 4, 2025
Merged

Upgrade Next version #70

merged 1 commit into from
Jan 4, 2025

Conversation

cjrace
Copy link
Owner

@cjrace cjrace commented Jan 4, 2025

Overview of changes

Fixes https://github.com/cjrace/almostyellow/security/dependabot/2 by upgrading packages. Had to do some fixing of the login after the upgrade and there was an error in boom boom pirate but all seems to be working now with these changes.

Checklist

  • I have tested these changes locally using pnpm test
  • I have updated the documentation (if needed)
  • I have added or updated tests for these changes (if applicable)

Reviewer instructions

Worth testing out whether you can login, log out, not access pages you shouldn't etc all still okay as you could before.

Also should definitely run the full suite of admin tests locally (as they don't run in GitHub actions).

Sorry, something went wrong.

@cjrace cjrace added the dependencies Pull requests that update a dependency file label Jan 4, 2025
@cjrace cjrace requested a review from lauraselby January 4, 2025 01:35
@cjrace cjrace merged commit ca5956f into main Jan 4, 2025
5 checks passed
@cjrace cjrace deleted the fix-2-next-upgrade branch January 4, 2025 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants