Skip to content
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

Overhaul sitemap #620

Closed
maiertech opened this issue Feb 22, 2023 · 0 comments · Fixed by #626
Closed

Overhaul sitemap #620

maiertech opened this issue Feb 22, 2023 · 0 comments · Fixed by #626
Labels

Comments

@maiertech
Copy link
Owner

maiertech commented Feb 22, 2023

  • I turned off the sitemap temporarily while working on Refactor posts.ts #612.
  • Before overhauling the sitemap, I double-checked how other devs do sitemaps with SvelteKit.

Sitemap overview

Do I need a sitemap?

Google specifics

https://developers.google.com/search/docs/crawling-indexing/sitemaps/build-sitemap#xml

  • You can submit an RSS feed as a sitemap.
  • Google ignores priority and changefreq.
  • Do you need sitemap extensions? Probably not.

Alternative

This comment proposes an alternative when using adapter-static: https://github.com/bartholomej/svelte-sitemap

  • Scans routes in build and creates build/sitemap.xml.
  • Uses the postbuild hook in package.json.

Approach

  • Build on endpoints I already have, e.g., posts.json, categories.json, and tags.json.
  • Add additional information to these endpoints.
  • Add certain static pages manually.
  • Add static paths as an array.
@maiertech maiertech changed the title Bring sitemap back Restore sitemap Feb 24, 2023
@maiertech maiertech changed the title Restore sitemap Overhaul sitemap Feb 24, 2023
maiertech added a commit that referenced this issue Feb 25, 2023
maiertech added a commit that referenced this issue Feb 25, 2023
maiertech added a commit that referenced this issue Feb 25, 2023
maiertech added a commit that referenced this issue Feb 25, 2023
maiertech added a commit that referenced this issue Feb 25, 2023
maiertech added a commit that referenced this issue Feb 25, 2023
@maiertech maiertech mentioned this issue Feb 25, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant