Skip to content

update analytics handling#767

Open
jameslamb wants to merge 2 commits intomainfrom
fix/google-analytics
Open

update analytics handling#767
jameslamb wants to merge 2 commits intomainfrom
fix/google-analytics

Conversation

@jameslamb
Copy link
Copy Markdown
Member

@jameslamb jameslamb commented Apr 9, 2026

Some changes to how consent for analytics cookies is handled on rapids.ai pages. I'll give more details privately.

Notes for Reviewers

How I tested this

Honestly, I'm very unsure how to test this. The deployment preview renders fine but:

  • will our Google Analytics stuff still work?
  • is this interacting with OneTrust correclty?

I'll need some help testing this (have asked privately).

@jameslamb jameslamb changed the title WIP: update analytics handling update analytics handling Apr 9, 2026
@jameslamb jameslamb marked this pull request as ready for review April 9, 2026 16:27
@jameslamb jameslamb requested a review from a team as a code owner April 9, 2026 16:27
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 9, 2026

Deploy Preview for docs-rapids-ai ready!

Name Link
🔨 Latest commit a867b07
🔍 Latest deploy log https://app.netlify.com/projects/docs-rapids-ai/deploys/69d7d2deebad190008704b12
😎 Deploy Preview https://deploy-preview-767--docs-rapids-ai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 9, 2026

Deploy Preview for docs-rapids-ai ready!

Name Link
🔨 Latest commit a453135
🔍 Latest deploy log https://app.netlify.com/projects/docs-rapids-ai/deploys/69d7d2f391ed970008bb2287
😎 Deploy Preview https://deploy-preview-767--docs-rapids-ai.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant