Skip to content

Docs: Restore legacy routes for migrated pages - #112655

Merged
Blargian merged 4 commits into
masterfrom
codex/restore-missing-legacy-doc-routes
Aug 1, 2026
Merged

Docs: Restore legacy routes for migrated pages#112655
Blargian merged 4 commits into
masterfrom
codex/restore-missing-legacy-doc-routes

Conversation

@dhtclk

@dhtclk dhtclk commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Restore eight legacy Docusaurus URLs for English pages recovered by the documentation migration audit. The pages and navigation are already present on master, but these post-snapshot legacy slugs were absent from the migration manifest and did not receive redirects to their canonical Mintlify routes.

This preserves existing public links for four ClickStack demo-day pages, Estuary Native, Tigris, the BYOC shared-responsibility page, and the custom-role RBAC migration guide.

Related: #112512
Related: ClickHouse/clickhouse-docs#6480
Related: ClickHouse/clickhouse-docs#6486
Related: ClickHouse/clickhouse-docs#6255
Related: ClickHouse/clickhouse-docs#6496
Related: ClickHouse/clickhouse-docs#6373
Related: ClickHouse/clickhouse-docs#6476
Related: ClickHouse/clickhouse-docs#6528
Related: ClickHouse/clickhouse-docs#6542

Changelog category (leave one):

  • Documentation (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Not applicable (documentation).

Version info

  • Merged into: 26.8.1.574 (included in 26.8 and later)

@clickhouse-gh

clickhouse-gh Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Workflow [PR], commit [435ac07]

Summary:


AI Review

Summary

This PR restores eight legacy docs redirects in docs/_site/redirects.json, but the ClickStack part is still incomplete: the English redirects for 2026-07-02 and 2026-07-10 were added without the matching legacy locale variants that the existing manifest already carries for ja, ko, ru, and zh. Some public legacy URLs therefore still 404, so this is not ready to merge as-is.

Findings

⚠️ Majors

  • [docs/_site/redirects.json:54307-54312] The PR promises to restore the missing legacy routes for pages already present on master, but the July 2 and July 10 ClickStack demo-day pages also ship localized variants in docs/{ja,ko,ru,zh}/clickstack/demo-days/2026/, and redirects.json already preserves the same locale-prefixed legacy ClickStack URLs through 2026-06-12. Without adding /jp|ko|ru|zh/use-cases/observability/clickstack/demo-days/2026/2026-07-{02,10} to the corresponding /<locale>/clickstack/demo-days/... destinations, those localized legacy URLs still 404. Suggested fix: add the eight missing locale redirect entries next to the new English ones, following the existing 2026-06-12 pattern (/jp/... -> /ja/..., others 1:1).
Final Verdict

Changes needed.

@clickhouse-gh clickhouse-gh Bot added the pr-documentation Documentation PRs for the specific code PR label Jul 30, 2026
@dhtclk
dhtclk requested a review from Blargian July 30, 2026 20:07
@dhtclk
dhtclk marked this pull request as ready for review July 30, 2026 20:07
Comment thread docs/_site/redirects.json
Comment on lines +54295 to +54300
"destination": "/clickstack/demo-days/2026/2026-07-02",
"source": "/use-cases/observability/clickstack/demo-days/2026/2026-07-02"
},
{
"destination": "/clickstack/demo-days/2026/2026-07-10",
"source": "/use-cases/observability/clickstack/demo-days/2026/2026-07-10"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The earlier ClickStack demo-day redirects restored the locale-prefixed legacy URLs as well (/jp, /ko, /ru, /zh), but this hunk only adds the English /use-cases/... sources. That leaves the July 2 and July 10 translated pages in an odd state: docs/{ja,ko,ru,zh}/clickstack/demo-days/2026/2026-07-{02,10}.mdx are already present and live under /ja|ko|ru|zh/clickstack/..., yet the old locale URLs still 404 (for example /ko/use-cases/observability/clickstack/demo-days/2026/2026-07-02). Please add the localized redirect entries for both dates as well, using /jp/... -> /ja/... for Japanese to match the existing pattern.

@mintlify

mintlify Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
ClickHouse-docs 🟢 Ready View Preview Jul 30, 2026, 8:26 PM

@dhtclk dhtclk changed the title Docs: Restore legacy routes for migrated pages Docs: Restore migrated routes and canonicalize BYOC overview Jul 31, 2026
@dhtclk dhtclk changed the title Docs: Restore migrated routes and canonicalize BYOC overview Docs: Restore legacy routes for migrated pages Jul 31, 2026
@Blargian
Blargian added this pull request to the merge queue Aug 1, 2026
Merged via the queue into master with commit a3d543d Aug 1, 2026
179 checks passed
@Blargian
Blargian deleted the codex/restore-missing-legacy-doc-routes branch August 1, 2026 06:52
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Aug 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-documentation Documentation PRs for the specific code PR pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants