PnX-SI / GeoNature

Application de saisie et de synthèse des observations faune et flore
GNU General Public License v3.0
104 stars 102 forks source link

Changement de `is_habitat_complex` #3125

Closed jacquesfize closed 2 weeks ago

jacquesfize commented 4 months ago

In order to fit the latest standard of the SINP, this PR propose to replace the Station attribute is_habitat_complex (boolean) with id_nomenclature_habitat_complexity.

camillemonchicourt commented 2 months ago

On parlait de "Complexes d'habitat" et pas de complexité d'habitat. Et dans la V2 du standard, "complexe" a été renommé "mosaïque" :

ComplexeHabitatValue -> renommée MosaiqueValue ; ajout de la valeur ‘4 : Mosaïque de type inconnu’.

Donc à voir si on part directement sur ce vocabulaire pour éviter de le changer quand on passera sur la V2 du standard.


Si j'ai bien suivi, le champ qui était un booléen précédemment n'était pas affiché en interface. Mais si certains l'ont renseigné dans la BDD, alors on perdra les données ? On ne peut pas les basculer sur la nomenclature ?

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.87%. Comparing base (b76a751) to head (e4c8aff). Report is 1 commits behind head on develop.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #3125 +/- ## ======================================== Coverage 83.87% 83.87% ======================================== Files 122 122 Lines 9647 9647 ======================================== Hits 8091 8091 Misses 1556 1556 ``` | [Flag](https://app.codecov.io/gh/PnX-SI/GeoNature/pull/3125/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=PnX-SI) | Coverage Δ | | |---|---|---| | [pytest](https://app.codecov.io/gh/PnX-SI/GeoNature/pull/3125/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=PnX-SI) | `83.87% <100.00%> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=PnX-SI#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

jacquesfize commented 2 weeks ago

prête pour le merge