Skip to content

fix(www): include showcase pages in sitemap - #1026

Closed
zigzagdev wants to merge 1 commit into
magicuidesign:mainfrom
zigzagdev:fix/www-sitemap-showcase
Closed

zigzagdev wants to merge 1 commit into
magicuidesign:mainfrom
zigzagdev:fix/www-sitemap-showcase

Conversation

@zigzagdev

Copy link
Copy Markdown
Contributor

Description

apps/www/app/sitemap.ts never imported showcaseSource, so the statically generated /showcase/[...slug] pages were missing from sitemap.xml while docs and blog pages were included.
This adds showcase pages to the sitemap using the same pattern as the existing docs/blog entries.

Changes

  • Import showcaseSource from @/lib/source in apps/www/app/sitemap.ts
  • Add showcaseSource.getPages() entries to the sitemap output (no lastModified, since the showcase frontmatter schema has no date field)

Motivation

/showcase/[...slug] pages are statically generated (generateStaticParams) and publicly linked from /showcase, but were excluded from sitemap.xml, so search engines had no sitemap-driven discovery path for them.
Verified via pnpm build: the generated sitemap.xml now contains all 26 /showcase/* URLs, matching the 26 files in content/showcase.

Breaking Changes

None.

Screenshots

N/A (sitemap XML output only)

Checklist

  • pnpm check passes locally
  • pnpm build passes locally
  • pnpm build:registry was run and the generated files are committed (if you changed registry/ or config/site.ts — CI verifies this on every PR)
  • PR title follows Conventional Commits (e.g., fix(www): include showcase pages in sitemap)

showcaseSource was never imported in sitemap.ts, so the statically
generated /showcase/[...slug] pages were excluded from search engine
indexing while docs and blog pages were included.
@vercel

vercel Bot commented Oct 13, 2026

Copy link
Copy Markdown

@zigzagdev is attempting to deploy a commit to the product-studio Team on Vercel.

A member of the Team first needs to authorize it.

@Yeom-JinHo

Copy link
Copy Markdown
Member

Thanks @zigzagdev, but this one is intentional. I dropped the showcase entries in #1015 (52bd34c) because the /showcase/[slug] detail pages are too thin (a title and one card) to promote for indexing. They're still linked from /showcase, so crawlers can reach them.

Closing.

@Yeom-JinHo Yeom-JinHo closed this Oct 14, 2026
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.

2 participants