MDAnalysis / UserGuide

User Guide for MDAnalysis
https://userguide.mdanalysis.org
22 stars 33 forks source link

fix: broken sitemap links #279

Closed jandom closed 1 year ago

jandom commented 1 year ago

Maybe fixes #172 – it worked for me locally (MacOS), and the sitemap.xml file looked sane – but I don't know enough about how this works to confidently say "this won't break something else"

sitemap.xml.txt

lilyminium commented 1 year ago

I found this... https://github.com/jdillard/sphinx-sitemap/issues/14 are we just using an old version of the extension?

Looks like that commit first looks for html_baseurl and falls back to site_url -- it's probably better to use the canonical config.

jandom commented 1 year ago

Nice, PTAL – using html_baseurl gave me a valid sitemap

jandom commented 1 year ago

Team work is dream work, thank you @lilyminium