aggrosoft / oxid-cookie-compliance

GDPR Compliance
MIT License
4 stars 9 forks source link

added vanilla javascript, added jquery switch, bunped dependecies, fi… #19

Closed ivoba closed 3 years ago

ivoba commented 3 years ago

We are heading to a post jquery time and themes like moga with bootstrap 5 already removed jquery dependency. So this switch to use module without jquery is introduced.

kingschnulli commented 3 years ago

The new mode switch defaults to false, this would change the behaviour for users updating to the new version. Could you please default this to the old behaviour? Everything else looks fine for merging

ivoba commented 3 years ago

@kingschnulli yes you are right. I changed default value to true.

ivoba commented 3 years ago

@kingschnulli Thank you. I just noticed i forgot to add agcookiecompliance-vanilla.min.js Will create a new PR.