Skip to content

feat: add OneTrust cookie consent, GTM, and privacy choices footer link#121

Draft
thisistonydang wants to merge 1 commit into
mainfrom
thisistonydang/cookies
Draft

feat: add OneTrust cookie consent, GTM, and privacy choices footer link#121
thisistonydang wants to merge 1 commit into
mainfrom
thisistonydang/cookies

Conversation

@thisistonydang

Copy link
Copy Markdown
Collaborator

What

  • Loads the standard Databricks OneTrust cookie consent scripts in the <head> of every page. Environment-aware: production variant on the production deployment, test variant on preview deployments, disabled in local dev (ONETRUST_ENV=test|production overrides for local testing).
  • Adds Google Tag Manager (same global container as www.databricks.com), loaded below the OneTrust tags and only when consent management is present; <noscript> fallback injected after <body>.
  • Adds the "Your Privacy Choices" footer link with the GPC icon to all page shells; it opens the OneTrust preference center. Unifies the legal-links list into src/lib/legal-links.ts.

Why

Brings the site's cookie consent and tag management in line with other Databricks web properties.

Design

Tag descriptors live in framework-neutral src/lib/ modules with thin Docusaurus shells (config headTags spread + one preBodyTags plugin), so the pending Next.js migration (#119) only re-renders them from the root layout.

Verification

Banner, preference center, and consent-gated GTM verified live in a browser; per-environment build output greps (prod IDs on production builds, zero tags on default builds); full unit + e2e suites pass.

This pull request and its description were written by Isaac.

@thisistonydang thisistonydang force-pushed the thisistonydang/cookies branch from 2d83521 to 84fbde2 Compare July 14, 2026 21:23
@thisistonydang thisistonydang marked this pull request as draft July 14, 2026 22:44
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