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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

reszli created an issue. See original summary.

Pooja Ganjage’s picture

FileSize
3.87 KB

Hi,

Creating a patch for this issue.

Please review the patch.

Thanks.

Pooja Ganjage’s picture

Status: Active » Needs review

rembrandx’s picture

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

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

Status: Fixed » Closed (fixed)

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

rembrandx’s picture

Version: 1.0.x-dev » 1.0.0-alpha4