-
Notifications
You must be signed in to change notification settings - Fork 26.9k
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
Updated with-sitemap example for App Router #66995
Updated with-sitemap example for App Router #66995
Conversation
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 |
@archanaagivale30 Hi there, one thing we should change is not using |
…ap-example-update
…ivale30/next.js into sitemap-example-update
Hi @samcx, |
@archanaagivale30 No need to merge canary—everything time you do, I have to override (or merge canary myself) so we successfully run the CI. |
@samcx understood I will take care of this with future changes. |
@archanaagivale30 Are you positive this file gets created on the development server inside a |
@samcx It will not create sitemap.xml file in /public folder as per documentation of App Router its implemented. It will create automatically on server side application and will accessible on route '/sitemap.xml'. And for Static site it will generate in out directory. |
Hello
This PR updates the with-sitemap example to use: