dnum-mi / vue-dsfr

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

Error on module import #244

Closed leihuayi closed 2 years ago

leihuayi commented 2 years ago

Dans un projet déjà existant en vue3 + vue-router4 où j'ai fait: 1.npm install @laruiss/vue-dsfr@beta

  1. Rajouté
    import VueDsfr from '@laruiss/vue-dsfr'            // Import (par défaut) de la bibliothèque
    import '@laruiss/vue-dsfr/dist/vue-dsfr-fonts.css' // Import des polices
    import '@laruiss/vue-dsfr/dist/vue-dsfr.css'       // Import des styles globaux

    et rajouté app.use(VueDsfr)

j'ai une erreur en ouvrant mon site:

./node_modules/@laruiss/vue-dsfr/dist/vue-dsfr.esm.js 954:31
Module parse failed: Unexpected token (954:31)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
laruiss commented 2 years ago

Bonjour,

Est-ce que vous utilisez Vite ?

beryl-koller commented 2 years ago

Bonjour, Non, on travaille sur un projet déjà existant qui avait été initié avec vue-cli en Vue2, qu'on a récemment migré en Vue3 pour pouvoir y intégrer le DSfr. Tout est public sur ce repo : https://github.com/datalab-mi/Basegun/tree/develop (branche develop).

beryl-koller commented 2 years ago

@laruiss petite information complémentaire (je continue à chercher !) : le module fonctionne jusqu'à la version 1.0.0-beta.14, puis j'ai cette erreur à partir de la version beta-16 (puisque pas de beta-15 disponible)

laruiss commented 2 years ago

C’est fortement déconseillé d’utiliser vue-cli qui utilise webpack. Essayez de migrer vers Vite.

laruiss commented 2 years ago

J’ai créé une PR pour migrer vers Vite https://github.com/datalab-mi/Basegun/pull/28

beryl-koller commented 2 years ago

Merci beaucoup @laruiss ! On avait finalement pris cette direction, et tout juste terminé la migration vers Vite en fin de matinée. Je vais comparer avec ta PR pour vérifier si on a rien loupé dans le processus.

laruiss commented 2 years ago

Je vous laisse fermer cette issue si tout fonctionne