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

start migrating to nextjs app router #1976

Merged
merged 1 commit into from Jan 27, 2024
Merged

start migrating to nextjs app router #1976

merged 1 commit into from Jan 27, 2024

Conversation

jdeniau
Copy link
Member

@jdeniau jdeniau commented Jan 27, 2024

NextJS introduces a new "app router" system in next. Now that we are on next 14, let's migrate the doc to the new system to be future-proof.

@jdeniau jdeniau marked this pull request as ready for review January 27, 2024 14:22
@jdeniau jdeniau merged commit 32de514 into main Jan 27, 2024
5 checks passed
@jdeniau jdeniau deleted the nextjs-app-router branch January 27, 2024 14:24
jdeniau added a commit that referenced this pull request Jan 27, 2024
* Upgrade eslint and ignore no-constructor-return rule for actual constructors (#1974)

* Upgrade eslint and ignore no-constructor-return rule for actual constructors

* ignore eslint-next for now

* upgrate documentation website to next 14 (#1975)

* upgrate to next 12

* next 13

* migrate links

* next 14

* deactivate eslint for documentation once again

* Fix title warning. See vercel/next.js#38256

* start migrating to nextjs app router (#1976)

* upgrade next sitemap (#1978)
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.

None yet

1 participant