Skip to content

add sitemap.xml generation during build#49

Merged
generall merged 2 commits intomainfrom
add-sitemap-generation
Apr 23, 2026
Merged

add sitemap.xml generation during build#49
generall merged 2 commits intomainfrom
add-sitemap-generation

Conversation

@qdrant-cloud-bot
Copy link
Copy Markdown

Summary

  • Add scripts/generate_sitemap.sh that finds all .md files in public/ and generates a standard sitemap.xml with URLs based on the site's base URL (defaults to https://qdrant-skills.netlify.app, overridable via SITE_URL env var).
  • Hook the script into build.sh so the sitemap is generated on every deploy.
  • Scope the text/markdown content-type header in netlify.toml to /*.md instead of /*, and add a dedicated application/xml header for /sitemap.xml.

Test plan

  • Verify bash build.sh produces public/sitemap.xml with all 27 skill URLs
  • Confirm Netlify deploy preview serves /sitemap.xml with Content-Type: application/xml
  • Confirm .md files still get Content-Type: text/markdown

Made with Cursor

* add scripts/generate_sitemap.sh that discovers all .md files and
  produces a standard sitemap.xml in public/
* call the script from build.sh after copying skills
* scope the text/markdown content-type header to /*.md so sitemap.xml
  is served as application/xml

Made-with: Cursor
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 23, 2026

Deploy Preview for qdrant-skills ready!

Name Link
🔨 Latest commit b861a95
🔍 Latest deploy log https://app.netlify.com/projects/qdrant-skills/deploys/69e9e8e94c579900080c7a63
😎 Deploy Preview https://deploy-preview-49--qdrant-skills.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@generall generall merged commit 41e13dc into main Apr 23, 2026
8 checks passed
@generall generall deleted the add-sitemap-generation branch April 23, 2026 09:50
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