-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
refactor(docs): migrate home page to app router #27221
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Ignored Deployments
|
No changes detected in Branching Preview Branches by Supabase. |
f768274
to
11ad6f9
Compare
11ad6f9
to
3004002
Compare
3004002
to
5a7b567
Compare
3184b5b
to
47b8d6f
Compare
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.
Nice. Checked all previews and everything looks good 👍
d4ccf78
to
41b00dc
Compare
ca6c611
to
24dd780
Compare
Checked all previews. Also tested log explorer on studio to make sure URL search params still work properly there |
Migrate docs home page to App Router.
Biggest changes touching shared packages are:
Need to put
use client
on a bunch of stuffNeed to switch out for
next/compat/router
(this prevents straight-out erroring if Pages Router isn't mounted, but doesn't provide a replacement for the actual functionality)Checked to make sure I haven't broken meta/OpenGraph
Checked responsive styles
Checked build size -- minor increase expected, nothing awful
Checked load time navigating between routers -- again minor increase and slight flicker expected, nothing awful
Checked for console errors -- only ones are 404s on the RSC prefetches, which are expected