Problem/Motivation

GTM can directly read the cookies, so eu_cookie_compliance_gtm is not a dependency anymore.
It's only needed if you wish to push some fancy computed data or additional values to GTM.

Proposed resolution

Remove the dependency form .info.yml
Clean up the default configs under /condfig/install

Remaining tasks

Create a patch.

CommentFileSizeAuthor
#2 3206915-2.patch3.87 KBPooja Ganjage

Comments

reszli created an issue. See original summary.

Pooja Ganjage’s picture

Pooja Ganjage commented
StatusFileSize
new 3206915-2.patch3.87 KB

Hi,

Creating a patch for this issue.

Please review the patch.

Thanks.

Pooja Ganjage’s picture

Pooja Ganjage commented
Status: Active » Needs review

rembrandx’s picture

rembrandx commented
Status: Needs review » Reviewed & tested by the community

I can confirm that it is no longer needed (corroborated by 2 other devs).
I tested the patch but had to re-add the config files (without the gtm related lines), as they are needed for category settings

rembrandx’s picture

rembrandx commented
Status: Reviewed & tested by the community » Fixed
rembrandx’s picture

rembrandx commented

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

rembrandx’s picture

rembrandx commented
Version: 1.0.x-dev » 1.0.0-alpha4