Skip to content

Comments

feat: update cookie hash for current language#2076

Merged
Su (sushmangupta) merged 3 commits intomainfrom
feat/update-cookie-hash-language-id
Feb 11, 2026
Merged

feat: update cookie hash for current language#2076
Su (sushmangupta) merged 3 commits intomainfrom
feat/update-cookie-hash-language-id

Conversation

@BrocksiNet
Copy link
Contributor

@BrocksiNet Björn Meyer (BrocksiNet) commented Jan 20, 2026

PR with changes: shopware/shopware#14264

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 Micha Hobert (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 Micha Hobert (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 Björn Meyer (@BrocksiNet) 👍

Feel free to merge if the next release is close.

@sushmangupta
Copy link
Contributor

Björn Meyer (@BrocksiNet) , thanks for the PR. :) The release has gone out, and hence, merging this.

@sushmangupta Su (sushmangupta) merged commit fbe290e into main Feb 11, 2026
8 checks passed
@sushmangupta Su (sushmangupta) deleted the feat/update-cookie-hash-language-id branch February 11, 2026 05:51
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