-
Notifications
You must be signed in to change notification settings - Fork 279
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
[sitecore-jss-nextjs/react][nextjs/react][v20] Update next to v13, react to 18 #1830
[sitecore-jss-nextjs/react][nextjs/react][v20] Update next to v13, react to 18 #1830
Conversation
… hotfix/20.0.0/jss-3453-update-next # Conflicts: # packages/create-sitecore-jss/src/templates/nextjs/package.json
… hotfix/20.0.0/jss-3453-update-next # Conflicts: # packages/sitecore-jss-nextjs/package.json # yarn.lock
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I could not find anything that is applicable here and missed comparing this to the React 18 and Nextjs13 upgrades done in the main branch. Maybe this should be marked as braking change and upgrade guide added. Some unit tests are failing, otherwise approved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great work. Looks good.
Is this backwards compatible for people/projects who are not yet in a position to run React 18 and Next 13? |
Description / Motivation
This PR updates nextjs to version 13. In addition to this:
Testing Details
Types of changes