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

Migrate to nextjs 15 #1036

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Migrate to nextjs 15 #1036

wants to merge 6 commits into from

Conversation

rin-st
Copy link
Member

@rin-st rin-st commented Jan 17, 2025

Next 15 what's new and migration guides here

codemods used:

react:
main migration codemod
types-react-codemod

nextjs:
async request api

Plus changed the next config file

Note: After updating deps, compile times for se-2 homepage were crazy 60-100 seconds, but then the problem eventually disappeared. Currently compile times in dev mode for me is like 8-13sec. Hopefully, for you it will work fine from the start


Things I thought about but decided it's not worth it


TODOs after migration to next15

  1. Update SRE extensions
  1. Update Eslint to v9 and all related packages, finally

Fixes #1033

@damianmarti
Copy link
Member

Thanks @rin-st I will take a look and test.

Maybe some of the extensions have to be updated too, mainly if they use a page param, right?

Copy link
Member

@damianmarti damianmarti left a comment

Choose a reason for hiding this comment

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

The code looks good and it seems do not have any missing migration stuff.

Tested and it's working fine.

The first time the compilation took me about 16 seconds and next one about 8 seconds, so it doesn't be an issue there.

I'm getting a deprecation warning after the compilation is done:

✓ Compiled / in 8.1s (7831 modules)
(node:142505) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.

Thanks!!

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.

Migrate to Nextjs v15
2 participants