Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding google analytics tag #148

Closed
wants to merge 2 commits into from
Closed

Adding google analytics tag #148

wants to merge 2 commits into from

Conversation

muralibasani
Copy link
Contributor

No description provided.

Signed-off-by: muralibasani <[email protected]>
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 30, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 86235d4
Status: ✅  Deploy successful!
Preview URL: https://f3212cbc.klaw-docs.pages.dev
Branch Preview URL: https://analytics.klaw-docs.pages.dev

View logs

@programmiri
Copy link
Contributor

@muralibasani there's a linting error it seems, should be fixable by running npm run reformat.

This should have popped up in the pre-commit hook 🤔 not sure why it seems not to have.

@programmiri
Copy link
Contributor

programmiri commented Oct 30, 2023

I don't think we should release that without setting up a cookie banner getting user consent and potential opt-out to be compliant with GDPR.

Google has a post for that: https://www.cookiebot.com/en/google-analytics-gdpr/

Aiven also has internal information, but I'm not sure what applies to us (bc different domain etc) so we should check that out, too.

Signed-off-by: Mirjam Aulbach <[email protected]>
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