Skip to content

RDoc-3857 Sitemap should contain only canonical URLs#2525

Open
poissoncorp wants to merge 1 commit into
ravendb:mainfrom
poissoncorp:sitemap-canonical-urls
Open

RDoc-3857 Sitemap should contain only canonical URLs#2525
poissoncorp wants to merge 1 commit into
ravendb:mainfrom
poissoncorp:sitemap-canonical-urls

Conversation

@poissoncorp

@poissoncorp poissoncorp commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Exclude non-current doc versions (7.1, 6.2, legacy) from the sitemap so it lists only the current canonical version's URLs. Stable sitemap-docs.xml naming and splitter cleanup are deferred to RDoc-3955.

@poissoncorp poissoncorp requested a review from Lwiel June 19, 2026 05:45
@Lwiel

Lwiel commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Does it mean we can have a single sitemap-docs.xml file instead of sitemap-docs-x.y.xml where x.y is latest version?

@poissoncorp poissoncorp force-pushed the sitemap-canonical-urls branch 2 times, most recently from 87e8cd8 to 0696893 Compare June 24, 2026 21:48
Non-current doc versions (active 7.1/6.2 and legacy) canonicalize to the
current version, so the sitemap must not list them. Exclude them via the docs
preset's sitemap ignorePatterns (active + legacy version paths), leaving only
the current version's canonical URLs.
@poissoncorp poissoncorp force-pushed the sitemap-canonical-urls branch from 0696893 to 2e3f99b Compare June 25, 2026 05:10
@poissoncorp

Copy link
Copy Markdown
Contributor Author

@Lwiel

Does it mean we can have a single sitemap-docs.xml file instead of sitemap-docs-x.y.xml where x.y is the latest version?

Yes, but we should wait with that for the next release, so we don't cause SEO lag & reindexing just for tidiness. I've thought about it for a while and reduced this PR to minimal changes while the rest (simplification) will be done here - https://issues.hibernatingrhinos.com/issue/RDoc-3955/Current-only-docs-sub-sitemap-with-stable-name

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