Skip to content

Conversation

@BrocksiNet
Copy link
Contributor

@BrocksiNet BrocksiNet commented Jan 20, 2026

PR with changes: shopware/shopware#14264

@BrocksiNet BrocksiNet requested a review from Isengo1989 January 20, 2026 08:07
@Isengo1989 Isengo1989 added the Undocumented Feature Feature which is not documented yet and is considered for documentation label Jan 21, 2026
@Isengo1989 Isengo1989 requested a review from Copilot January 21, 2026 08:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the cookie consent management documentation to reflect that the cookie configuration hash is now stored per-language rather than as a single value. This enables proper consent tracking when multiple languages are served from the same domain.

Changes:

  • Added detailed explanation of hash storage format as a language ID-to-hash object
  • Updated workflow descriptions to clarify per-language hash comparison
  • Added info boxes explaining domain and language handling behavior

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
guides/plugins/plugins/storefront/add-cookie-to-manager.md Added info box explaining hash storage format and updated workflow steps to reflect per-language hash tracking
concepts/commerce/content/cookie-consent-management.md Updated sequence diagram, mechanism details, Store API description, and table to reflect per-language hash storage and comparison

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Isengo1989 Isengo1989 force-pushed the feat/update-cookie-hash-language-id branch from 82c4ba5 to 6c6fb1c Compare January 21, 2026 14:34
Copy link
Contributor

@Isengo1989 Isengo1989 left a comment

Choose a reason for hiding this comment

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

Thx for the PR @BrocksiNet 👍

Feel free to merge if the next release is close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Undocumented Feature Feature which is not documented yet and is considered for documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants