-
Notifications
You must be signed in to change notification settings - Fork 102
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
[MONITORING] Questionnements sur le dossier backend/geonature/core/gn_monitoring #2197
Comments
Ça me semble en effet plus logique … Il me semble que si le schéma |
Top merci @bouttier pour ton retour ! Je procèderai au déplacement à part si quelqu'un s'y oppose |
Non, pas forcément. Ensuite la module générique "Monitoring" a été développé, mais il y a aussi les modules de suivi de la flore qui s'appuient dessus, et peut-être d'autres. L'idée était plutôt que certains ajouts à A rediscuter avec le PnCévennes. |
Je confirme que l'objectif est plutôt de centraliser le modèle L'idée d'avoir aussi un bloc |
Contexte
Dans le cadre d'une prestation pour le MNHN, l'entrée par site dans le module
gn_monitoring
est en train d'être développée.Question
En voulant ajouter une colonne (
id_categorie
) àt_base_sites
, viagn_monitoring
, il s'avère que le modèle correspondantTBaseSites
est présent dans GeoNature et non dansgn_monitoring
(https://github.com/PnX-SI/GeoNature/tree/master/backend/geonature/core/gn_monitoring).Ce dernier ainsi que
TBaseVisits
ont l'air d'être utilisés uniquement dansgn_monitoring
.Pourrait-on déplacer ces modèles dans
gn_monitoring
?Éléments de réponses
Il semblerait que l'objectif futur serait de ramener la gestion des sites au sein de GeoNature. Doit-elle être faite dans un module
GN_Sites
comme mentionné par #1263 ?Mais en attendant, et vu que la prestation impose que la gestion des sites soit faite dans
gn_monitoring
, cela semble donc plus logique de déplacer ces modèles dansgn_monitoring
pour lui en laisser la responsabilité. Qu'en pensez-vous ?The text was updated successfully, but these errors were encountered: