dnum-mi / vue-dsfr

Portage en Vue du DSFR
https://vue-ds.fr
Other
61 stars 28 forks source link

Erreur avec webpack : not found: Error: Default condition should be last one #573

Closed ajacque-trydea closed 1 year ago

ajacque-trydea commented 1 year ago

Bonjour, Depuis la release 4.0.1, il est impossible d'utiliser le package dans une application construite par webpack.

A la compilation, l'erreur suivante est lancée : Module not found: Error: Default condition should be last one

Après quelques recherches, il semble que dans la partie "exports" de package.json, "default" devrait être le dernier élément de la liste. Or dans les fichiers de release, "types" intervient après "default"

refs : doc webpack doc node

Comment reproduire le bug :

initialiser un projet node avec le fichier package.json suivant :

{
  "name": "untitled",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "build": "webpack"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "devDependencies": {
    "webpack": "^5.88.1",
    "webpack-cli": "^5.1.4"
  },
  "dependencies": {
    "@gouvminint/vue-dsfr": "^4.1.0"
  }
}

créer un fichier src/index.js avec le contenu suivant :

import VueDsfr from '@gouvminint/vue-dsfr'

executer npm run build

Contournement du problème :

Utiliser la version 4.0.0 du package vue-dsfr

modifier le package.json du package dans node module pour placer les "export.default" en dernière position (non reecommandé, uniquement pour devs)

laruiss commented 1 year ago

Bonjour, merci, je vais regarder ça très bientôt.

Par curiosité, pourquoi utilisez-vous encore webpack ?

ajacque-trydea commented 1 year ago

Merci d'avance,

Le projet tourne sur une base de php74 + symfony et la gestion des assets est plus simple avec webpack (bien que sa configuration soit un calvaire). De plus il s'agit d'un projet historiquement sous webpack. Donc peu de possibilité de changements

github-actions[bot] commented 1 year ago

:tada: This issue has been resolved in version 4.2.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

laruiss commented 1 year ago

@ajacque-trydea Bonjour, voudrez-vous bien confirmer que c’est corrigé dans la v4.2.1 ?

ajacque-trydea commented 1 year ago

Je confirme que tout fonctionne normalement, merci pour la reactivité !