-
-
Notifications
You must be signed in to change notification settings - Fork 246
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
docs: Update docs and examples to Next.js 15 #1089
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This reverts commit a8496ab.
…nings by making sure the middleware doesn't import from anything React-specific
# Conflicts: # examples/example-app-router-playground/package.json # examples/example-app-router-playground/src/components/ClientLink.tsx # examples/example-app-router-playground/src/components/NavigationLink.tsx # examples/example-app-router-playground/src/navigation.ts # examples/example-app-router/package.json # pnpm-lock.yaml
# Conflicts: # examples/example-app-router-playground/package.json # examples/example-app-router-playground/src/components/ClientLink.tsx # examples/example-app-router-playground/src/components/NavigationLink.tsx # examples/example-app-router-playground/src/i18n/request.tsx # examples/example-app-router-playground/src/middleware.ts # examples/example-app-router-playground/src/navigation.ts # examples/example-app-router/package.json # pnpm-lock.yaml
# Conflicts: # docs/pages/docs/getting-started/app-router/with-i18n-routing.mdx # examples/example-app-router-mixed-routing/src/app/(public)/[locale]/about/page.tsx # examples/example-app-router-mixed-routing/src/app/(public)/[locale]/page.tsx # examples/example-app-router/src/app/[locale]/page.tsx # examples/example-app-router/src/app/[locale]/pathnames/page.tsx
…g server-only apis
Update: I'll wait with updating docs and examples for Next.js 15 for the time being since there's a critical bug in Next.js 15 that affects multilingual apps that use i18n routing. However, I've added a separate PR that merely bumps the peer dependency: #1443. |
Earlier I created a new issue #1587 but didn't realize you were already working on a larger housekeeping task to update the entirety of the docs. This PR will fix my newly reported issue as well. Just mentioning it here to keep track of possible duplicates like mine 😅 |
@ChristianIvicevic Thanks for the note! Yeah, I hope Next.js addresses the upstream issues that were introduced in Next.js 15 so I can start recommending it without drawbacks! :) |
Fixes #1587
TODO
next/font
interferes with header links from middleware. vercel/next.js#69000 (comment)not-found
page interferes with static rendering vercel/next.js#71623