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

Copy link
Copy Markdown
Contributor

Issue link

RDoc-3857 Sitemap should contain only canonical URLs

Additional description

Non-current doc versions (active 7.1/6.2 and legacy) canonicalize to the current version, so listing them in the sitemap contradicts their own canonical tag. Exclude every built version except current via the docs preset's sitemap ignorePatterns, and switch the post-build sitemap splitter from a legacy denylist to a current-version allowlist so both layers enforce the same invariant.

Type of change

  • Content - docs
  • Content - cloud
  • Content - guides
  • Content - start pages/other
  • New docs feature (consider updating /templates or readme)
  • Bug fix
  • Optimization
  • Other

Changes in docs URLs

  • No changes in docs URLs
  • Articles are restructured, URLs will change, mapping is required (update /scripts/redirects.json file, set Documents Moved PR label)

Changes in UX/UI

  • No changes in UX/UI
  • Changes in UX/UI (include screenshots and description)

Non-current doc versions (active 7.1/6.2 and legacy) canonicalize to the
current version, so listing them in the sitemap contradicts their own
canonical tag. Exclude every built version except current via the docs
preset's sitemap ignorePatterns, and switch the post-build sitemap splitter
from a legacy denylist to a current-version allowlist so both layers enforce
the same invariant.
@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?

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