Skip to content

docs(generic-writer): retire the dev Generic Writer pages, redirect to help (PRDCT-553) - #414

Merged
Iamfle4ka merged 2 commits into
mainfrom
docs/retire-generic-writer
Aug 26, 2026
Merged

docs(generic-writer): retire the dev Generic Writer pages, redirect to help (PRDCT-553)#414
Iamfle4ka merged 2 commits into
mainfrom
docs/retire-generic-writer

Conversation

@Iamfle4ka

Copy link
Copy Markdown
Collaborator

DO NOT MERGE before keboola/connection-docs#1052 — the redirects would 404.

All 3 pages move to help.keboola.com under Components › Writers (keboola/connection-docs#1052) at mirrored subpaths, so they become redirect_to stubs — same shape as #406/#412: plain redirects, no banner.

  • /extend/generic-writer/**https://help.keboola.com/components/writers/generic-writer/**
  • no images/assets were exclusive to these pages; nothing to delete
  • Generic Writer subtree dropped from _data/navigation.yml
  • no staying page links into the subtree, so no link rewrites needed

Note: _data/navigation.yml will trivially conflict with the sibling Generic Extractor retirement (#413) — the two nav blocks are adjacent; whichever merges second needs a one-line rebase.

🤖 Generated with Claude Code

…o help (PRDCT-553)

All 3 pages moved to help.keboola.com under Components › Writers in
keboola/connection-docs#1052 at mirrored subpaths, so they become
`redirect_to` stubs, same shape as #406/#412.

- /extend/generic-writer/** -> https://help.keboola.com/components/writers/generic-writer/**
- no images/assets were exclusive to these pages; nothing to delete
- the Generic Writer subtree is dropped from _data/navigation.yml
- no staying page links into the subtree, so no link rewrites needed

Note: _data/navigation.yml will trivially conflict with the sibling
Generic Extractor retirement (PRDCT-552) — the two blocks are adjacent;
whichever merges second needs a one-line rebase.

DO NOT MERGE before keboola/connection-docs#1052 — the redirects would 404.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@linear-code

linear-code Bot commented Aug 25, 2026

Copy link
Copy Markdown

PRDCT-553

@davidesner davidesner left a comment

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.

LGTM

@Iamfle4ka
Iamfle4ka marked this pull request as ready for review August 26, 2026 17:23

@keboola-pr-reviewer-bot keboola-pr-reviewer-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Verdict: needs_human (risk 3/5) · profile docs

Needs human: nav-data edit, page retirements, and unverifiable cross-repo redirect targets all require review.

Impact flags: possible rollback re-introduction — see Check Run summary.

Concerns:

  • _data/navigation.yml: Navigation structure edited; policy requires human review for _data/ changes.
  • extend/generic-writer/index.md: redirect_to targets help.keboola.com; cross-repo existence unverifiable from diff.
  • extend/generic-writer/configuration/configuration.md: Redirects 404 until connection-docs#1052 merges; merge-order hazard.

Suggested reviewers: @keboola/docs

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
Iamfle4ka merged commit be07bc0 into main Aug 26, 2026
1 check passed
@Iamfle4ka
Iamfle4ka deleted the docs/retire-generic-writer branch August 26, 2026 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants