Enable IAB: Script missing attributes

Here's a patch for adding data-tcf attributes when enabling IAB via module settings.

Hi @aelfendir!

Thank you a lot for the issue and patch.

I checked the documentatiuon for IAB in Cookie information help pages at https://support.cookieinformation.com/en/articles/8331323-how-to-use-the....

It seems like your patch solves it partly but according to the document the change in the popup is following:
<script id="CookieConsent" type="text/javascript" src="https://policy.app.cookieinformation.com/uc.js" data-culture="EN" data-tcf-v2-enabled="true" data-tcf-version="2.2"> </script>

Currently the patch does not seem to take the second data attribute data-tcf-version into account.

Here's a new patch that follows the updated Cookie Information documentation.

Thanks a lot @aelfendir.

This looks good to me know. We don't currently have any project with IAB support so I cannot further test if this is working as it should. Do you have any information to share how this could be verified?

That being said, the actual code looks good and I would be willing to mark this as RTBC.

I also noticed that the link I posted earlier is now giving an error 404. This seems to work:
https://support.cookieinformation.com/en/articles/8331323-how-to-use-the....

There is also a setting for this in the documentation:
data-tcf-global-scope="false"---> ENABLES GLOBAL VS. LOCAL SCOPE OF THE CONSENT STRING

But I think this would need a new configuration option in the module settings to toggle this as true/false but it could be a separate issue to change IAB scope.

According to IAB Europe, global scope has been deprecated in IAB TCF version 2.0. The reference to global scope appears to me as outdated information. Current documentation is for IAB TCF 2.2.

Testing the consent would most likely require you to implement a script from an IAB partner vendor and activate the IAB mode as per Cookie Information's instructions.

Looks good now. I will mark this as RTBC. We will check when we can merge this.

Can @aelfendir you make a merge request for this?

Fixed conflict and merged. Thanks @aelfendir.

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