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

[frontend] Suppression du fichier app.config.ts (?) #2747

Closed
3 tasks done
jacquesfize opened this issue Oct 10, 2023 · 4 comments
Closed
3 tasks done

[frontend] Suppression du fichier app.config.ts (?) #2747

jacquesfize opened this issue Oct 10, 2023 · 4 comments
Assignees
Labels
frontend javascript Pull requests that update Javascript code solved in next release

Comments

@jacquesfize
Copy link
Contributor

jacquesfize commented Oct 10, 2023

Je suis en train de cartographier le backend et frontend de Geonature. Au cours de mon voyage (:smile:), je suis tombé sur le fichier app.config.ts (dans le dossier frontend/src/conf) qui contient une version JSON du fichier de configuration de Geonature (config/geonature_config.toml). Selon @TheoLechemia , ce fichier n'est plus censé être utilisé, ni généré. Est-ce bien le cas ?

Si oui, je propose de :

  • Vérifier si des modules tierces de Geonature utilisent encore ce fichier
  • Modifier le programme d'installation de Geonature pour supprimer la génération du fichier app.config.ts
  • Supprimer le template présent dans frontend/src/conf
@jacquesfize jacquesfize added frontend javascript Pull requests that update Javascript code labels Oct 10, 2023
@jacquesfize jacquesfize self-assigned this Oct 10, 2023
@TheoLechemia
Copy link
Member

Merci !
Il y a aussi un tour à faire côté backend pour supprimer les fonction / commandes geonature qui se chargeaient de créer ce fichier. Notamment : https://github.com/PnX-SI/GeoNature/blob/master/backend/geonature/utils/command.py#L22 et https://github.com/PnX-SI/GeoNature/blob/master/backend/geonature/core/command/main.py#L75

@Pierre-Narcisi
Copy link
Contributor

Je pense qu'il faudra aussi modifier la CI pour enlever la partie où on l'imprime : https://github.com/PnX-SI/GeoNature/blob/master/.github/workflows/cypress.yml#L106

@camillemonchicourt
Copy link
Member

En lien avec #2205 qui prévoyait cette suppression dans la 2.13.
Ça n'a pas été fait dans la 2.13, donc ça sera dans la 2.14.
A bien répercuter dans les modules comme expliqué dans #2205.
C'est fait dans Import, Export, Monitoring, Dashboard, et maintenant Zones humides.

@camillemonchicourt
Copy link
Member

Fichier supprimé dans la 2.14.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend javascript Pull requests that update Javascript code solved in next release
Projects
None yet
Development

No branches or pull requests

4 participants