-
Notifications
You must be signed in to change notification settings - Fork 26.7k
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
Update blog-starter
to App Router
#61170
Conversation
f2dc77e
to
19cec62
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.
Thank you for submitting a
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
cfc8e6c
to
05c9389
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.
Head branch was pushed to by a user without write access
8ec9626
to
315dc9b
Compare
Can I have some guidance on how to get all of the Checks to pass? |
@zeevo Only maintainers are allowed to approve CI runs so I went ahead and approved your commit again. There was an issue with the |
@samcx Woot! Thank you for the merge |
What?
This updates the
blog-starter
example to Next 14.1 App Router.Why?
I checked out a new
blog-starter
project on Vercel and was surprised it was using the Pages Router. I believe the App Router is a better choice.How?
I tried to keep the implementation logic as close to the original Pages
blog-starter
.