Skip to content

fix(seo): add sitemap-blog.xml route handler + include in sitemap index (BUY-57939) - #234

Open
BuyWhere wants to merge 1 commit into
mainfrom
fix/BUY-57939-sitemap-blog
Open

fix(seo): add sitemap-blog.xml route handler + include in sitemap index (BUY-57939)#234
BuyWhere wants to merge 1 commit into
mainfrom
fix/BUY-57939-sitemap-blog

Conversation

@BuyWhere

Copy link
Copy Markdown
Owner

Fix for BUY-57939

Problem: /sitemap-blog.xml returns 404 — no route handler exists for a dedicated blog sitemap. The sitemap index at /sitemap.xml also doesn't reference it.

Changes:

  • src/app/sitemap-blog.xml/route.ts (NEW) — Blog sitemap route handler using getAllBlogPosts(), emits all 22 blog posts with proper lastmod, changefreq, and priority
  • src/app/sitemap.xml/route.ts — Added sitemap-blog.xml to the sitemap index

Verification (pre-fix live check):

Endpoint Status
/sitemap.xml 200 (index — will include blog after deploy)
/sitemap-pages.xml 200 (231 URLs)
/sitemap-blog.xml 404 (before fix) → 200 (after deploy)
/sitemap-products.xml 200 (203 URLs)
/sitemap-products-sg.xml 410 (intentional)
/robots.txt 200
/sitemap-categories.xml 200
/sitemap-compare.xml 200

Deploy: Requires Railway deployment after merge.

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.

1 participant