docs(variables): retire the dev Variables pages, redirect to help (PRDCT-554) - #412
Merged
Conversation
…DCT-554)
The two /integrate/variables/ pages moved to help.keboola.com under Components,
per Jordan's 08-05 placement call and Matyáš's 08-07 confirmation in
#ask-component-factory ("Call it Variables and I don't mind wherever it is.
Components as a page is fine.").
- /integrate/variables/ -> https://help.keboola.com/components/variables/api/
- /integrate/variables/tutorial/ -> https://help.keboola.com/components/variables/api/tutorial/
- assets deleted; the help pages carry their own copies
- both entries dropped from the Integrate nav
Same stub shape as the merged /integrate/mcp/ retirement (#405).
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
…ables # Conflicts: # integrate/variables/index.md # integrate/variables/tutorial.md
Iamfle4ka
marked this pull request as ready for review
August 25, 2026 11:12
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: needs_human (risk 3/5) · profile docs
Escalating: nav-data edit, two page retirements, and asset deletions all trip always-needs-human rules.
Impact flags: possible rollback re-introduction — see Check Run summary.
Concerns:
_data/navigation.yml: Navigation data changed; drives site-wide sidebar rendering — always requires human review.integrate/variables/index.md: Live page gutted to redirect stub; page retirement can break inbound links.integrate/variables/countries.csv: Four assets deleted; remaining references unverifiable (code search unavailable).- + 1 more concern — see Check Run summary.
Suggested reviewers: @keboola/docs
This was referenced Aug 25, 2026
Merged
davidesner
self-requested a review
August 25, 2026 15:30
Iamfle4ka
pushed a commit
that referenced
this pull request
Aug 26, 2026
Brings the branch up to date with main (#412/#413 retirements). No conflicts — this branch only touches _layouts/redirect.html. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wc7aH2VDs4qYwxCai1JuSA
Iamfle4ka
pushed a commit
that referenced
this pull request
Aug 26, 2026
Resolves 5 conflicts, all the same case the PR description already handles for extend/component/**: main has since turned these pages into redirect_to stubs (#412 Variables, #413 Generic Extractor), so the audit fixes queued for them change nothing a reader can see on this site. Main's stub wins in each: extend/generic-extractor/configuration/api/pagination/index.md stray ] after [EX139] extend/generic-extractor/configuration/aws-signature/index.md "signaturing"/"extractor sign" wording extend/generic-extractor/configuration/config/index.md garbled userData sentence extend/generic-extractor/configuration/ssh-proxy/index.md "you tu" typo; stray period after [EX131] integrate/variables/index.md 2 invalid-JSON fixes (missing , and ") Those 7 fixes now belong against help, alongside connection-docs#1077 — they are listed above so they are not lost. Everything else in this PR is untouched. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wc7aH2VDs4qYwxCai1JuSA
Iamfle4ka
pushed a commit
that referenced
this pull request
Aug 26, 2026
Resolves 10 conflicts. Every one is on a page main has since turned into a redirect_to stub (#405/#406/#412/#413), so the fix queued for it is invisible to readers of this site; main's stub wins in each case: extend/common-interface/config-file.md "colummns"; missing , after destination extend/common-interface/development-branches.md "accross" -> "across" extend/common-interface/folders.md URL split across a newline; missing leading /; "my-inpupt" extend/component/running/index.md "you loose"; 2 heading renames for inbound links .../generic-extractor/.../api/pagination/index.md offset link pointed at the pagenum page .../generic-extractor/configuration/config/index.md #dataType -> #data-type extend/generic-extractor/functions.md unterminated "http://example.com/ string extend/generic-extractor/tutorial/basic.md #configuration-sections anchor; missing , in JSON integrate/mcp.md 2 root-relative links that pointed off-site integrate/variables/index.md singular #...-for-variable anchor; localhost:4000 URL Those belong against help now, alongside connection-docs#1077 — listed above so they are not lost. The 11 surviving fixes (/cli/, /integrate/jobs/, /overview/, /extend/generic-writer/) are untouched. 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
3 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.
What & why
Second half of the two-PR shape for the Variables unit (PRDCT-583). The content is already migrated to help — this retires the dev-side originals so there is exactly one maintained copy.
/integrate/variables//integrate/variables/tutorial/Placement and naming are owner-decided, not proposed here:
#ask-component-factory: "Call it Variables and I don't mind wherever it is. Components as a page is fine." There is no separate Variables API product — the calls are configuration/Storage API plus Vault.What changed
redirect_tostubs — the same shape as the already-merged/integrate/mcp/retirement (docs(mcp): retire the dev MCP page, redirect to help (PRDCT-580) #405).countries.csv,variables.svg, two tutorial screenshots); the help pages carry their own copies, verified indist.Deliberately not in this PR
_layouts/redirect.html— the friendly "this page has moved" template lands with docs(extend): retire the merged component docs, redirect to help (PRDCT-543) #406. Until thenjekyll-redirect-fromuses its own template, so the redirect works either way; no need to add the same file in two open PRs.extend/component/index.md:94, the only inbound link from outside this subtree, is left pointing at/integrate/variables/— it redirects, and docs(extend): retire the merged component docs, redirect to help (PRDCT-543) #406 turns that whole file into a stub anyway. Repointing it here would only conflict with docs(extend): retire the merged component docs, redirect to help (PRDCT-543) #406.Merge order
Draft on purpose: this must not merge before connection-docs#1051, which publishes the target pages. #1051 is green and mergeable, waiting on review.
🤖 Generated with Claude Code