docs(storage): retire the dev Storage API pages, redirect to help (PRDCT-551) - #416
Merged
Conversation
…DCT-551) All 10 pages moved to help.keboola.com under Storage in keboola/connection-docs#1053 (API pages at mirrored subpaths, client libraries under /storage/api/clients/), so they become `redirect_to` stubs — same shape as #406/#412/#413/#414. - /integrate/storage/{,api/**} -> https://help.keboola.com/storage/api/** - /integrate/storage/<lang>-client/ -> https://help.keboola.com/storage/api/clients/* (docker-cli-client -> clients/docker-cli — the one renamed slug) - 3 assets used only by these pages deleted (help carries its own copies) - the Storage subtree dropped from _data/navigation.yml - 9 links from staying pages (/, /integrate/, /overview/, /overview/api/) now point at help directly - links from /integrate/variables/** and /extend/generic-*/** left as-is: those subtrees retire in #412/#413/#414; they work via stubs meanwhile DO NOT MERGE before keboola/connection-docs#1053 — the redirects would 404. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
davidesner
self-requested a review
August 25, 2026 15:31
Iamfle4ka
marked this pull request as ready for review
August 26, 2026 16:55
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: needs_human (risk 3/5) · profile docs
Coordinated Storage-docs retirement touching site nav and adding unverifiable cross-repo redirects; multiple policy backstops require a human.
Impact flags: possible rollback re-introduction — see Check Run summary.
Concerns:
_data/navigation.yml: Sidebar nav Storage subtree removed; always-needs-humanintegrate/storage/index.md: redirect_to help.keboola.com targets unverifiable, may 404 before #1053integrate/storage/docker-cli-client.md: Renamed slug docker-cli-client to clients/docker-cli needs confirmation
Suggested reviewers: @keboola/docs
Both pages stay on this site and this PR only rewrites 6 Storage links in them, but they were also normalised CRLF -> LF, which rewrote every line: 351 changed lines instead of 6. That buried the real change in review and made both files collide with every other open PR that touches them (#410, #402), since no line merged cleanly. Content is byte-identical to the previous commit — line endings only, back to what main has. The 8 integrate/storage/** files are left alone: they become redirect stubs here, so their whole content is replaced regardless. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wc7aH2VDs4qYwxCai1JuSA
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
DO NOT MERGE before keboola/connection-docs#1053 — the redirects would 404.
All 10 pages move to help.keboola.com under Storage (keboola/connection-docs#1053): API pages at mirrored subpaths, client libraries under
/storage/api/clients/. Same stub shape as #406/#412/#413/#414 — plain redirects, no banner./integrate/storage/{,api/**}→https://help.keboola.com/storage/api/**/integrate/storage/<lang>-client/→…/storage/api/clients/*(docker-cli-client →clients/docker-cliis the one renamed slug)_data/navigation.yml/,/integrate/,/overview/,/overview/api/) now point at help directly/integrate/variables/**and/extend/generic-*/**left as-is on purpose — those subtrees retire in docs(variables): retire the dev Variables pages, redirect to help (PRDCT-554) #412/docs(generic-extractor): retire the dev Generic Extractor pages, redirect to help (PRDCT-552) #413/docs(generic-writer): retire the dev Generic Writer pages, redirect to help (PRDCT-553) #414; rewriting them here would only manufacture merge conflicts🤖 Generated with Claude Code