[FEAT] Ajoute vue vers le SINP National#238
[FEAT] Ajoute vue vers le SINP National#238k-patrinat wants to merge 27 commits intoPnX-SI:masterfrom
Conversation
Ajoute un script pour les transmissions vers le SINP National et une note d'utilisation détaillant le contenu de la requête et les limites
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
|
OK super, merci pour cette contribution. A voir aussi si on l'ajoute à tous dans une migration ou si on l'indique dans la doc pour ceux qui veulent. |
Je suis d'accord pour la renommer, d'autant plus que c'est bien une vue temporaire et qu'il faut l'identifier comme telle. Ta proposition me convient. |
ahurabielle
left a comment
There was a problem hiding this comment.
Merci pour la proposition de vue. Cela me paraît vraiment souhaitable d'uniformiser nos pratiques.
Je préconise vraiment l'ajout du champ date_validation
|
|
||
| Il a été décidé, dans la mesure où la synthèse ne comporte qu'un seul identifiant unique pour une donnée (le second | ||
| étant lié à un type de regroupement) que les trois niveaux `Evenement`, `SujetObservation` et `Descriptif` récupèrerait | ||
| ce même identifiant pour permettre une traçabilité de la donnée sur ces trois niveaux. |
There was a problem hiding this comment.
Autant, je comprends ce choix pour SujetObservation et Descriptif vu les précédentes discussions / choix de traduction par les organismes.
Autant, je trouve cela dommage pour Evenement car en pratique, dans les modules OccTax et monitoring, ce qui est renseigné dans l'attribut unique_id_sinp_grp correspond en pratique au concept d'évenement.
On pourrait donc faire le choix de le récupérer s'il est non NULL ou le mettre à la valeur d'unique_id_sinp si NULL pour ne pas laisse ce champ vide.
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
backend/gn_module_export/migrations/data/Notice utilisation v_export_inpn.md
Outdated
Show resolved
Hide resolved
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #238 +/- ##
=============================
=============================
☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
…export_inpn.md Co-authored-by: ahurabielle <adele@hurabielle.com>
d5376af to
f49715b
Compare
f49715b to
7b411ed
Compare
|
Le PN-Pyrénées avait fait un travail de projection GN -> SOTv3 assez intéressant, cité dans le ticket GeoNature#3285 On y trouvait notamment une fonction pour faire la traduction entre les nomenclatures GN (possiblement modifiées/enrichies selon les instances) et la nomenclature standard SOTv3. Cette fonction, qui prend la valeur source et le champ cible en argument, permet de gérer les cas où plusieurs champs GN peuvent alimenter le même champ SOTv3 (plusieurs appels dans un L'objectif n'est pas forcément le même cela dit. On avait une vue par niveau attendu dans le nouveau standard plutôt qu'une projection à plat. On aurait donc une vue événement, une autre SujetObs, une troisième descSujet... et donc autant d'exports / d'API différentes. |
…l'identifiant de l'Evenement
Ajoute un script pour les transmissions vers le SINP National et une note d'utilisation détaillant le contenu de la requête et les limites