Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problème Leaflate.Deflate #21

Open
jpm-cbna opened this issue May 27, 2020 · 6 comments
Open

Problème Leaflate.Deflate #21

jpm-cbna opened this issue May 27, 2020 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@jpm-cbna
Copy link
Collaborator

jpm-cbna commented May 27, 2020

Le module SHT utilise le plugin Leaflet.Deflate pour améliorer l'affichage des points sur la carte quelque soit le niveau de zoom. Or avec la version 2.3 de GeoNature (Angular 7) ce plugin en version 1.3.0 provoque l'erreur suivante :

ERROR in ../external_modules/sht/frontend/node_modules/Leaflet.Deflate/types/index.d.ts(14,34): error TS2694: Namespace '"/home/jpm/workspace/geonature/web/geonature/frontend/node_modules/@types/leaflet.markercluster/index"' has no exported member 'MarkerClusterGroupOptions'.

Les dépendances définies dans le package.json étaient:

"dependencies": {
    "@types/leaflet": "^1.4.3",
    "@types/leaflet.markercluster": "^1.4.0",
    "leaflet": "~1.3.1",
    "leaflet.markercluster": "^1.4.1",
    "Leaflet.Deflate": "^1.3.0"
  }

L'installation de "@types/leaflet" et "@types/leaflet.markercluster" suppriment des erreurs liées à TypeScript mais il reste malgré tout l'erreur ci-dessus.

La seule solution supprimant l'erreur et l'ajout directement dans GeoNature du plugin Leaflet.Deflate à l'aide de la commande: npm install Leaflet.Deflate

Nous allons ajouter le plugin comme dépendance de GeoNature en attendant de trouver une meilleur solution au problème ci-dessus.

@jpm-cbna jpm-cbna added the bug Something isn't working label May 27, 2020
@jpm-cbna jpm-cbna self-assigned this May 27, 2020
@camillemonchicourt
Copy link
Member

OK dans tous les cas c'est intéressant car ce plugin peut servir dans d'autres modules de GeoNature.

@camillemonchicourt
Copy link
Member

A voir car dans la 2.10.0 de GeoNature le plugin a été supprimé car non utilisé dans GeoNature.

@jpm-cbna
Copy link
Collaborator Author

Ok mais dans ce cas là, il faut fournir un moyen fonctionnel pour qu'un module externe puisse ajouter ses propres packages Node... Il me semble que ce n'était pas si évident...

@jpm-cbna jpm-cbna reopened this Oct 16, 2022
@TheoLechemia
Copy link
Member

Simplement ajouter un package.json dans le frontend du module. Le module d'import et dashboard le font

@camillemonchicourt
Copy link
Member

Le nettoyage : PnX-SI/GeoNature#2019
Idéalement, il est recommandé aussi d'avoir un fichier package-lock.jsonau niveau du module.

@jpm-cbna
Copy link
Collaborator Author

Le support des fichiers package.json semble être dispo pour la v2.9.2 et il a été amélioré dans la v2.10.0. Il faut donc tester l'ajout de ce fichier...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

No branches or pull requests

3 participants