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 cookie consent to theme, by default #86

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AshleyHenry15
Copy link
Contributor

What:

New sites cannot have google analytics or tracking unless we add cookie consent.
Quarto has cookie consent functionality.

I've added the functionality to the theme so it will be there by default.

We are waiting on approval from Rick - once that happens, I will merge this in.

Closes #81

Dependencies (hold-don't merge)

On hold/blocked by: https://github.com/rstudio/docs.rstudio.com/pull/1934

Testing steps

PRs will not be approved if testing notes aren't included here.

  • Build quarto docs
  • Google Inspect > Application tab > deleted all cookies
  • Added cookie consent
  • Rebuilt/previewed the docs
  • Cookie banner displays and also checked the Application tab to confirm that cookies are not present until I agree/opt in
2024-12-11_12-26-07

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.

Google Analytics
1 participant