Skip to content

fix: attribute official Discord links in rendered docs articles - #301

Merged
hannesrudolph merged 3 commits into
mainfrom
feat/docs-body-discord-tags-20261104
Nov 4, 2026
Merged

hannesrudolph merged 3 commits into
mainfrom
feat/docs-body-discord-tags-20261104

Conversation

@hannesrudolph

Copy link
Copy Markdown
Member

What Problem This Solves

Translated docs article links still bypass placement attribution even though the shared Discord navigation is tagged.

User Impact

Official plain Discord invite links in rendered articles use the first-party redirect with the canonical page placement, independently of language. Link text, support wording, code examples, event-specific invitations, and generated source content stay unchanged.

Why This Change Was Made

Apply the migration in the canonical article HTML renderer after its cache, rather than rewriting mirrored English or generated translations. FAQ, updating, and building-plugins use fixed page tags; other official article contexts use the registered generic docs tag.

Evidence

  • 298 local tests passed, including generated English/French HTML, cache reuse, Markdown/raw anchors, negative destinations, and existing browser analytics/privacy checks.
  • A real French preview showed zero direct official article anchors on FAQ, updating, and building-plugins, with the intended page tags.
  • Completed pinned local ClawSweeper review found no actionable code findings.
  • Merge is held until the new FAQ/updating placement keys are confirmed live at the destination; remote docs CI must also pass.

@hannesrudolph
hannesrudolph requested a review from a team as a code owner November 4, 2026 07:19
@hannesrudolph
hannesrudolph merged commit ca38dbc into main Nov 4, 2026
4 checks passed
@hannesrudolph
hannesrudolph deleted the feat/docs-body-discord-tags-20261104 branch November 4, 2026 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants