Problem/Motivation

In context of Drupal CMS we came to the conclusion that we disable the YouTube and Vimeo services by default out-of-the-box, and enable them automatically with an ECA model when the first media item will be created that uses either provider. For details see #3493442: Klaro should be inactive until Vimeo or Youtube content is added.

This means, that Klaro will be installed by default, but no consent management is required as only 2 apps are enabled that do not require the user to consent.

Proposed resolution

We should get Klaro into an inactive mode until at least 1 app is enabled that requires the user consent.

Issue fork klaro-3493540

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

jurgenhaas created an issue. See original summary.

jurgenhaas’s picture

Status: Active » Needs review

Seems pretty straightforward. Maybe it should be made configurable, I'm not sure.

catch’s picture

Manually tested by applying the MR diff against a stock Drupal CMS install. I then disabled the youtube and vimeo apps via the config page.

When logged out, only 5.6kb of JavaScript (compressed) instead of over 70kb compressed.

The footer link still shows up and now doesn't go anywhere, but I assume that's a separate issue?

catch’s picture

Status: Needs review » Reviewed & tested by the community

This makes sense to me and I don't really see a good case for configuration - can already configure which apps are enabled. Moving to RTBC.

jurgenhaas’s picture

The footer link still shows up and now doesn't go anywhere, but I assume that's a separate issue?

For new installations, the recipe got updated so that the menu link would be disabled when importing it as a new content entity. For testing this in an existing Drupal CMS installation you would have to manually disable not only the Klaro apps but also that menu link.

jan kellermann made their first commit to this issue’s fork.

jan kellermann’s picture

Status: Reviewed & tested by the community » Fixed

Thank you for the solution and the merge request.
I fixed small things and merged.

In general there is a problem with caching but I will open a new issue for this.

Status: Fixed » Closed (fixed)

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