Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update sitemap.mdx #66950

Merged
merged 1 commit into from
Jun 17, 2024
Merged

Update sitemap.mdx #66950

merged 1 commit into from
Jun 17, 2024

Conversation

KeisukeNagakawa
Copy link
Contributor

Fix: Correct URL generation in sitemap to use product.id

  • Updated sitemap function to correctly use product.id for URL generation instead of id.
  • Ensured each product's URL is accurately reflected in the sitemap output.

Fix: Correct URL generation in sitemap to use product.id

- Updated sitemap function to correctly use product.id for URL generation instead of id.
- Ensured each product's URL is accurately reflected in the sitemap output.
@KeisukeNagakawa KeisukeNagakawa requested review from a team as code owners June 17, 2024 08:13
@KeisukeNagakawa KeisukeNagakawa requested review from manovotny and leerob and removed request for a team June 17, 2024 08:13
@ijjk ijjk added the Documentation Related to Next.js' official documentation. label Jun 17, 2024
@ijjk
Copy link
Member

ijjk commented Jun 17, 2024

Allow CI Workflow Run

  • approve CI run for commit: a8e0aed

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

Copy link
Member

@samcx samcx left a 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 PR!

:lgtm:

@samcx samcx enabled auto-merge (squash) June 17, 2024 08:15
@samcx samcx merged commit cf75497 into vercel:canary Jun 17, 2024
32 checks passed
@github-actions github-actions bot added the locked label Jul 1, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 1, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Documentation Related to Next.js' official documentation. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants