fix: exports for bundlers #45
Conversation
✅ Deploy Preview for dsfr-chart ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Avez-vous pu regarder la PR svp ? @QuentiumYT @zellerbaptiste |
|
Cela me semble correct, mais l'utilisation du dossier bundler n'est pas nécessaire. Nous allons proposer un fix pour l'import des composants uniques je pense prochainement. @zellerbaptiste on peut faire une release minor si cela est nécessaire |
| ### Export du composant | ||
|
|
||
| Finalement déclarer le nouveau composant dans la section `exports` du `package.json` : | ||
|
|
||
| ```json | ||
| { | ||
| "exports": { | ||
| "./NewChart": { | ||
| "import": "./dist/bundler/NewChart.js", | ||
| "require": "./dist/bundler/NewChart.cjs" | ||
| } | ||
| } | ||
| } | ||
| ``` | ||
|
|
There was a problem hiding this comment.
Ne sera pas nécessaire avec un wildcard
"exports": {
"./*": {
"import": "./dist/*/*.js",
"require": "./dist/*/*.umd.cjs"
},
"./*/css": "./dist/*/*.css",
"./*.js": {
"import": "./dist/*/*.js",
"require": "./dist/*/*.umd.cjs"
},
"./*.css": "./dist/*/*.css",
"./dist/*/*": {
"import": "./dist/*/*.js",
"require": "./dist/*/*.umd.cjs"
},
"./dist/*/*/css": "./dist/**/*.css",
"./dist/*/*.js": {
"import": "./dist/*/*.js",
"require": "./dist/*/*.umd.cjs"
},
"./dist/*/*.css": "./dist/*/*.css"
},There was a problem hiding this comment.
Prenez également en compte la nouvelle configuration de vite proposé pour le build.
Sans cette configuration, il existe en effet aujourd'hui des builds pour:
- intégrer l'intégralité de tous les composants
- ou intégrer unitairement un seul composant
Cependant chaque build unitaire intègre ça propre version de vue et donc si on souhaite intègrer uniquement 2 graphiques, en intégrant les 2 builds unitaires des graphiques on se retrouve à charger 2 fois vue.
There was a problem hiding this comment.
Je suis d'accord avec cela, mais ne sachant pas comment les exports fonctionnent lorsqu'il y a une multitude de fichiers split, nous avons gardé le comportement de base. Je vais investiguer prochainement.
Proposition pour corriger le problème remonté dans l'issue: #42