sitemap #41
sitemap #41
Conversation
sitemap.xml
Outdated
There was a problem hiding this comment.
You don't need to upload the sitemap to git. It should be automatically generated.
sitemap/index.html
Outdated
There was a problem hiding this comment.
Do we really need this? I don't see how this will be automatically updated.
There was a problem hiding this comment.
Yes, this sitemap.html page is not automatically updated. I added it to help users find pages that don’t have direct navigation links elsewhere on the site. I’ve seen similar user-friendly sitemaps on many websites to improve discoverability.
I’m not sure how useful it will be, but I can explore ways to automate its updates if you think it’s worth keeping.
There was a problem hiding this comment.
There must be a way to generate such an HTML file alongside the XML that you are generating.
There was a problem hiding this comment.
yes, there is. But i wanted external links (eg. link to saiman project) also to be included in the sitemap, generating html file alongside the xml will not include these external links. But i'll see if there is a way. I'll update this.
There was a problem hiding this comment.
We don't need another CI action. Besides, it will not even work because it is not going to consider generated files at all (?). This functionality is available in Jekyll and should be enabled in the configuration.
There was a problem hiding this comment.
Thanks for the feedback! Just to clarify — since our site isn’t Jekyll-based, we can’t directly use jekyll-sitemap. Were you suggesting we switch to a Jekyll setup, or just pointing to that repo as a reference?
Currently, I have a Node.js script (generate-sitemap.js) that builds sitemap.xml automatically. I can make sure it avoids committing the sitemap to the repo.
Let me know if that sounds good, or if you had a different workflow in mind for non-Jekyll sites :)
There was a problem hiding this comment.
Haha sorry -- I forgot we are not using Jekyll. This makes sense.
There was a problem hiding this comment.
But yes, it would be good to have a solution that injects the sitemap to GitHub Pages without committing it to the repository.
There was a problem hiding this comment.
yes sure, i will update this so that it doesn't commit the sitemap to the repo.
|
Added Jekyll configuration to enable automatic sitemap generation
|
The generate-sitemap script now automatically updates sitemap.xml. I’ve also added a sitemap section to our site for user navigation, which includes pages that don’t have direct links elsewhere on the website. @kevalpithadiya @mrigankpawagi