milieuinfo / webcomponent-vl-ui-cookie-consent

1 stars 0 forks source link

[BUG] - vl-cookie-consent.min.js importeert niet vl-cookie-consent-opt-in.min.js #41

Closed driesva closed 4 years ago

driesva commented 4 years ago

Omschrijf het probleem vl-cookie-consent.min.js gebruikt niet de minified vl-cookie-consent-opt-in.min.js maar vl-cookie-consent-opt-in.js waardoor wij volgende warnings krijgen:

vl-form-label werd reeds gedefinieerd als custom element vl-core.min.js:115:35 vl-form-validation-message werd reeds gedefinieerd als custom element vl-core.min.js:115:35 vl-form-annotation werd reeds gedefinieerd als custom element vl-core.min.js:115:35 vl-checkbox werd reeds gedefinieerd als custom element

Relevant stuk van vl-cookie-consent.min.js dat aantoont dat niet de minified versie wordt gebruikt:

import{VlElement,define}from"/node_modules/vl-ui-core/dist/vl-core.min.js";import"/node_modules/vl-ui-button/dist/vl-button.min.js";import"/node_modules/vl-ui-form-grid/dist/vl-form-grid.min.js";import"/node_modules/vl-ui-modal/dist/vl-modal.min.js";import{analytics}from"/node_modules/vl-ui-cookie-consent/src/analytics.js";import"/node_modules/vl-ui-cookie-consent/dist/vl-cookie-consent-opt-in.js";export class ...

Gewenst gedrag Overal minified versie.

Omgeving (gelieve aan te vullen):

coemans commented 4 years ago

Probleem zit op vl-ui-util niveau cfr. issue #160.