docs(generic-extractor): retire the dev Generic Extractor pages, redirect to help (PRDCT-552) - #413
Merged
Merged
Conversation
…rect to help (PRDCT-552) All 39 pages moved to help.keboola.com under Components › Data Source Connectors in keboola/connection-docs#1054 at mirrored subpaths, so they become `redirect_to` stubs, same shape as the merged /extend/ retirement (#406) and the Variables one (#412). - /extend/generic-extractor/** -> https://help.keboola.com/components/extractors/generic-extractor/** - every existing `redirect_from` is preserved (iterations, publish), so legacy URLs keep working in two hops - 47 images/assets used only by these pages are deleted; the help pages carry their own copies - the Generic Extractor subtree is dropped from _data/navigation.yml - 7 links from pages that stay (/, /extend/, CLI getting-started) now point at help directly instead of bouncing through a stub - links from /extend/generic-writer/** are left as-is on purpose: those pages retire in the sibling PRDCT-553 PR, rewriting them here would only manufacture a merge conflict; they work via the stubs meanwhile DO NOT MERGE before keboola/connection-docs#1054 — the redirects would 404. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This was referenced Aug 25, 2026
davidesner
self-requested a review
August 25, 2026 15:29
Iamfle4ka
marked this pull request as ready for review
August 26, 2026 15:54
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: needs_human (risk 4/5) · profile docs
PR exceeds size cap (2917 source LOC across 30 source files; 0 test/fixture LOC across 0 files excluded from the cap).
Iamfle4ka
pushed a commit
that referenced
this pull request
Aug 26, 2026
Resolves the _data/navigation.yml conflict flagged in the PR description: main removed the Generic Extractor nav subtree (#413), this branch removes the adjacent Generic Writer subtree — both go, so /extend/ is left with no children and the now-empty `items:` key is dropped with them. extend/index.md: the note claiming "Generic Extractor and Generic Writer are still documented on this site" is false once both retirements are in; it now points both at help.keboola.com/components/. 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
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.
DO NOT MERGE before keboola/connection-docs#1054 — the redirects would 404.
All 39 pages move to help.keboola.com under Components › Data Source Connectors (keboola/connection-docs#1054) at mirrored subpaths, so they become
redirect_tostubs — same shape as the merged /extend/ retirement (#406) and Variables (#412): plain redirects, no banner./extend/generic-extractor/**→https://help.keboola.com/components/extractors/generic-extractor/**redirect_fromentries preserved (iterations, publish) — legacy URLs keep working in two hops_data/navigation.yml/,/extend/, CLI getting-started) now point at help directly/extend/generic-writer/**left as-is on purpose — those pages retire in the sibling PRDCT-553 PR; rewriting them here would only manufacture a merge conflict🤖 Generated with Claude Code