- Issue created by @jan kellermann
- Merge request !76Call applyConsents while proceed() to load optOut scripts. → (Open) created by jan kellermann
- 🇩🇪Germany jan kellermann
I added applyConsents() while initialization.
Please review and feedback.
- 🇩🇪Germany jan kellermann
Bump. Updated fork, ready to review. Would be great to have this issue reviewed and merged for next release.
- 🇩🇪Germany sascha_meissner Planet earth
I reviewed this issue with the following result:
I can reproduce the issue and solve it by just setting "Toggled by default" as well.
The codechange in drupal.js is unnecessary and just leads to execute the script twice on load.
I´ll update the MR accordingly. - 🇩🇪Germany sascha_meissner Planet earth
After looking again over this together with Jan Kellermann
We found out that it does not work for elements with contextual consent elements, the ressource gets loaded but the contextual element is still shown.
So i´m setting this back to needs work ...! - 🇩🇪Germany sascha_meissner Planet earth
FYI: I just tested the vanilla library and have the same result, the contextual element is shown instead of the resource
Created an upstream issue https://github.com/klaro-org/klaro-js/issues/536 - 🇩🇪Germany jan kellermann
This is fixed in https://github.com/klaro-org/klaro-js/issues/536
We are considering to update drupal/klaro-js.
- 🇩🇪Germany jan kellermann
See https://www.drupal.org/project/klaro_js/issues/3508237 🐛 Bugfix for #536: Check for optOut state while contextual rendering Active