Skip to content

Conversation

@wannevancamp
Copy link
Contributor

@wannevancamp wannevancamp commented Feb 2, 2026

@wannevancamp wannevancamp changed the title update cookie consent docs fix: update cookie consent docs Feb 2, 2026
@wannevancamp wannevancamp force-pushed the Add-Cookie-to-Manager branch from 523692f to 6dd45c3 Compare February 3, 2026 06:35
```

This will eventually lead to a new group being created, containing two new cookies, as well as a new cookie without a group.
This will add your cookie to the existing "Comfort Features" group in the cookie consent manager.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are many plugins that create their own consent groups. To avoid uncontrolled growth of consent groups, this example has been updated to show how you can add a cookie to an existing group.

@mitelg mitelg requested a review from BrocksiNet February 3, 2026 08:27
@Isengo1989 Isengo1989 added the Faulty Documentation PR created to correct some flaws in the documentation label Feb 10, 2026
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

Updates the Shopware plugin guide for registering custom cookies in the storefront cookie consent manager to reflect the Shopware 6.7+ event-based API and structured cookie objects.

Changes:

  • Replace references to CookieGroupsCollectEvent with CookieGroupCollectEvent.
  • Update the services.xml example to register the new event class.
  • Replace the PHP example to add a custom CookieEntry into the existing “Comfort Features” cookie group using the new structs/collection approach.

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

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

Labels

Faulty Documentation PR created to correct some flaws in the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants