-
Notifications
You must be signed in to change notification settings - Fork 8
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
[TTAHUB-2652] upgrade to nodejs 20 #2148
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Matt Bevilacqua <[email protected]>
Co-authored-by: Matt Bevilacqua <[email protected]>
Co-authored-by: Matt Bevilacqua <[email protected]>
Co-authored-by: Matt Bevilacqua <[email protected]>
…tart-TTADP into TTAHUB-2652/nodejs-20
@GarrettEHill I think the Modal component has changed enough to need to be re-implemented. It's unclear what else might need to be updated on the frontend. Switching to this branch and installing would take more than 30 minutes on my setup so it's hard to dip in and out of it. Since the updates are involved, this might be a case for getting everything else in place and writing a separate ticket to update the frontend that can be handled either this sprint or next. Unless you want to give doing the Modal over a go. |
Description of change
How to test
Issue(s)
Checklists
Every PR
Before merge to main
Production Deploy
After merge/deploy