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

ajout de geoman dans les dépendances npm #2115

Closed
wants to merge 1 commit into from

Conversation

joelclems
Copy link
Contributor

reprise de sur la partie geoman de la PR #2099

(la PR #2114 permet de faire en sorte que les markers s'affichent correctement avec geoman)

@joelclems
Copy link
Contributor Author

C'est un ajout de geoman dans les paquets npm de geonature
c'est destiné à être utilisé dans un module, mais je n'ai pas réussi à m'en servir correctement avec une instalation depuis un module

voir #2099 (comment)

@codecov
Copy link

codecov bot commented Nov 8, 2022

Codecov Report

Base: 64.13% // Head: 63.97% // Decreases project coverage by -0.16% ⚠️

Coverage data is based on head (ac79201) compared to base (8ebe38e).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2115      +/-   ##
==========================================
- Coverage   64.13%   63.97%   -0.17%     
==========================================
  Files          73       73              
  Lines        6365     6381      +16     
==========================================
  Hits         4082     4082              
- Misses       2283     2299      +16     
Flag Coverage Δ
pytest 63.97% <ø> (-0.17%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
backend/geonature/utils/command.py 0.00% <0.00%> (ø)
backend/geonature/core/admin/admin.py 77.77% <0.00%> (ø)
backend/geonature/core/command/main.py 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@bouttier
Copy link
Contributor

bouttier commented Nov 8, 2022

J’ai testé et je reproduis bien le soucis. Il vient de la mauvaise architecture du frontend, dans laquelle le dossier node_modules des modules GeoNature n’est pas un sous-dossier du dossier frontend de GeoNature.

Cette mauvaise architecture arrive à être quand même relativement fonctionnelle grâce au chemin absolue vers le node_modules de GeoNature dans le fichier tsconfig.json. Mais la suppression de la templétisation de ce fichier nous oblige à corriger cette architecture et c’est ce qui sera fait dans la prochaine version de GeoNature.

J’ai pu tester sur la branche feat/docker qui a cette nouvelle architecture : l’installation de geoman au niveau du module fonctionne correctement.

@joelclems
Copy link
Contributor Author

joelclems commented Feb 3, 2023

ça marche bien (je viens de voir ce post),
j'ai juste un soucis pour intégrer le fichier ss sans passer par le fichier angular.json

edit
il me manquait npm install external_modules/<nom du module> et ça marche

@joelclems joelclems closed this Feb 3, 2023
@camillemonchicourt camillemonchicourt deleted the feat/frontend-geoman branch February 3, 2023 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants