grrr-amsterdam / cookie-consent

Cookie consent with accessible dialog, agnostic tag triggers and conditional content, script and embed hooks.
MIT License
64 stars 11 forks source link

Convert cookie-consent to custom-element #28

Closed CountNick closed 7 months ago

CountNick commented 1 year ago

Convert the cookie-consent to a custom element

The cookie-consent package is now turned into a custom-element, there's still some code in here which might be redundant also the test needs to be updated probably. To test this feature out this branch needs to be linked with this Ammodo branch

So what's been done:

CountNick commented 1 year ago

@harmenjanssen ik heb bovenstaande punten verwerkt, ik kwam er alleen wel achter dat de constructor functie niet word uitgevoerd op safari, chrome incognito en firefox. Het zal dus ook op andere browsers wel niet werken op dit moment. Ik ben er nog niet achter waarom de constructor functie niet word uitgevoerd.

CountNick commented 1 year ago

@harmenjanssen @MartijnNieuwenhuizen Alles werkt nu naar behoren, er gaat nu een check op de test niet goed, maar ik weet niet hoe ik deze nu moet aanpassen.

MartijnNieuwenhuizen commented 1 year ago

@CountNick This PR can be closed right?