Skip to content

fix: remove analytics prompts from documentation - #256

Merged
hannesrudolph merged 1 commit into
mainfrom
fix/docs-remove-analytics-prompts
Oct 2, 2026
Merged

hannesrudolph merged 1 commit into
mainfrom
fix/docs-remove-analytics-prompts

Conversation

@hannesrudolph

@hannesrudolph hannesrudolph commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

What Problem This Solves

Removes the automatic Google Analytics prompt, all GA-added public privacy disclosures/copy, and the footer on/off control from public documentation.

User Impact

No replacement prompt appears. Existing valid saved grants continue to work; saved denials, GPC/DNT, expiry, storage failures and private-surface protections remain enforced. New visitors and visitors without a valid grant stay unmeasured, including notice/opt-out regions. This deliberately reduces analytics coverage.

Why This Change Was Made

The renderer stops emitting the analytics panel and footer button. Consent initialization continues without those elements and only accepts a valid existing explicit grant. Nonvisible helpers remain to keep this urgent change small. The legal/footer HTML is byte-for-byte identical to pre-GA base 5d328f31a30736208972cddd6445bdf3005afc59 (parent of the rollout merge), preserving existing Foundation/legal content. No standalone privacy link or public privacy page was created by the GA work; no destination is invented. All authored/translated docs and unrelated edits remain intact.

Evidence

  • Full Node suite: 274 passed, one optional SDK test skipped.
  • Focused desktop/mobile, light/dark checks verify no analytics prompt or controls and no Google request/cookie for visitors without a grant.
  • Existing saved-grant, denial, GPC/DNT, expiry, async-boundary and BFCache checks remain in the normal suite.
  • Bounded preview built successfully. Structured Autoreview is clean; Docs Code CI and CodeQL are the normal pre-merge checks.

No fresh full SDK recertification or forwarded GA QA is part of this UI removal.

@hannesrudolph
hannesrudolph requested a review from a team as a code owner October 2, 2026 22:10
@hannesrudolph
hannesrudolph merged commit 66edaff into main Oct 2, 2026
5 checks passed
@hannesrudolph
hannesrudolph deleted the fix/docs-remove-analytics-prompts branch October 2, 2026 22:18
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.

1 participant