Skip to content

Conversation

cderv
Copy link
Collaborator

@cderv cderv commented Oct 10, 2025

Doc for quarto-dev/quarto-cli#13525

This PR details the cookie consent logic with analytics-event: true and the requirement for cookie-consent: true

@cderv cderv requested a review from cwickham October 10, 2025 12:10
@github-actions github-actions bot temporarily deployed to pull request October 10, 2025 12:17 Inactive
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1806.quarto.org

🔄 Modified Documents

Copy link
Collaborator

@cwickham cwickham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Comments are all about what the default behaviour for cookie-consent: true is. I don't want us to imply that cookies are opt-in until that is true. We either wait until the defauly behavior is changed, or change examples to explicitly set type: express.


- **Default behavior (no cookies)**: A random session token is generated for each browser session. This allows tracking of search interactions during a single visit without storing persistent cookies. This default is compliant with privacy regulations like GDPR.

- **Persistent tracking (with cookies)**: To track users across multiple sessions, you must enable [cookie consent](website-tools.qmd#cookie-consent). When cookie consent is enabled, Algolia will only use cookies after the user grants "tracking" consent:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... after the user grants "tracking" consent

This won't be true until the default behavior for cookie-consent: true is type: express. We could omit, or wait to merge.

Or maybe:

Suggested change
- **Persistent tracking (with cookies)**: To track users across multiple sessions, you must enable [cookie consent](website-tools.qmd#cookie-consent). When cookie consent is enabled, Algolia will only use cookies after the user grants "tracking" consent:
- **Persistent tracking (with cookies)**: To track users across multiple sessions, you must enable [cookie consent](website-tools.qmd#cookie-consent). When cookie consent is enabled, Algolia will use cookies based on settings for `cookie-consent`:

analytics-events: true
```

Note that with cookie consent enabled, both Google Analytics and Algolia search analytics will only use cookies after the user has granted "tracking" consent. Without cookie consent enabled, Algolia will use session-only tracking without cookies. See [Algolia Insights](/docs/websites/website-search.qmd#algolia-insights) for more details.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

after the user has granted "tracking" consent

Same problem here.

@cderv
Copy link
Collaborator Author

cderv commented Oct 13, 2025

The default type change is in there:

Probably worth a 1.9 highlight ? or just breaking change bullet in changelog ?

@cscheid
Copy link
Collaborator

cscheid commented Oct 14, 2025

Probably worth a 1.9 highlight ? or just breaking change bullet in changelog ?

Definitely needs to be a 1.9 highlight.

@cderv cderv changed the title document new cookie behavior for algolia document new cookie behavior: type: express as defaumt, and new algolia behavior Oct 14, 2025
@cderv
Copy link
Collaborator Author

cderv commented Oct 14, 2025

@cwickham I have done the update in this PR regarding new 'express' default. I also added a highlight.

I tried to rework (with Claude's help) some of the wording to mention the regulation-ready without being too much about compliance, as we don't want to definitive on this.

@cderv cderv changed the title document new cookie behavior: type: express as defaumt, and new algolia behavior document new cookie behavior: type: express as default, and new algolia behavior Oct 14, 2025
@github-actions github-actions bot temporarily deployed to pull request October 14, 2025 15:45 Inactive
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1806.quarto.org

🔄 Modified Documents

@cderv cderv requested a review from cwickham October 14, 2025 15:47
Copy link
Collaborator

@cwickham cwickham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cderv
Copy link
Collaborator Author

cderv commented Oct 15, 2025

Thank you !

@cderv cderv merged commit f2bcba0 into prerelease Oct 15, 2025
3 checks passed
@cderv cderv deleted the algolia-cookie-opt-in branch October 15, 2025 10:02
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1806.quarto.org

🔄 Modified Documents

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.

3 participants