Skip to content

feat(docs): ask before Google Analytics sets cookies on the docs - #3286

Merged
datlechin merged 1 commit into
mainfrom
docs/analytics-consent
Oct 8, 2026
Merged

datlechin merged 1 commit into
mainfrom
docs/analytics-consent

Conversation

@datlechin

Copy link
Copy Markdown
Member

Why

docs.tablepro.app loads GA4 with no consent step. One page view sets _ga and _ga_GQYNPKSK83 on .tablepro.app, the parent domain. tablepro.app's privacy policy says Google Analytics cookies are not set until the reader allows them, so that sentence was false for anyone who had opened the docs.

Change

  • GA4 stays. integrations.cookies in docs.json holds all Mintlify analytics back until local storage has tablepro:analytics-consent = granted.
  • New docs/consent.js: the same question, wording and equal Allow and Decline buttons as tablepro.app, plus a "Cookie settings" control in the footer to change the answer.
  • With no answer or a Decline, the docs stream cookie set before this change is removed.

Verified

Headless Chrome against mint dev: first visit and after Decline send no request to Google and set no cookie; after Allow the page reloads and GA loads. No overlap with the assistant launcher at 1280 and 390 wide, light and dark.

Not in this PR

@mintlify

mintlify Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
TablePro 🟢 Ready View Preview Oct 8, 2026, 1:05 PM

💡 Tip: Enable Automations to automatically generate PRs for you.

@datlechin
datlechin merged commit 2072720 into main Oct 8, 2026
13 checks passed
@datlechin
datlechin deleted the docs/analytics-consent branch October 8, 2026 13:06

This branch was successfully deployed

1 active deployment
staging - docs — a788ff73 Deployed Oct 8, 2026 by mintlify[bot]
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