-
Notifications
You must be signed in to change notification settings - Fork 196
Migrate to Next.js App Router #315
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
base: main
Are you sure you want to change the base?
Migrate to Next.js App Router #315
Conversation
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
The /blog route 404s, that seems like a bug |
Co-Authored-By: [email protected] <[email protected]>
Co-Authored-By: [email protected] <[email protected]>
📝 Changed routes:
And 4 other routes: 10 deleted routes:
|
Did we ensure that global files like _app and _document were also considered? For the GTM stuff you might want to use @next/third-parties: https://nextjs.org/docs/app/guides/third-party-libraries Scan the pages/ dir and make sure we migrate all routes. I see there's a store route for security notice, for example, that didn't get moved over. The goal should be to delete pages/ entirely here. |
…routes Co-Authored-By: [email protected] <[email protected]>
Co-Authored-By: [email protected] <[email protected]>
…ocumentation Co-Authored-By: [email protected] <[email protected]>
Next.js App Router Migration
This PR migrates the hyper-site repository from Next.js pages router to app router and updates all dependencies to the latest versions.
Changes
Testing
next build
Link to Devin run
https://app.devin.ai/sessions/1938faa728824d40beef9360372d521f
Requested by: [email protected]