search

PnX-SI / Nomenclature-api-module

Flask module for Nomenclature API
GNU General Public License v3.0
3 stars 10 forks source link

Feature/ginco2-38 : Ajouter une colonne type jeux de données #31

Closed FloVollmer closed 4 years ago

TheoLechemia commented 4 years ago

Merci pour la PR. A noter qu'à chaque fois qu'on fait des modifs de la base, il faut bien les reporter dans les fichiers de migrations: voir cet exemple: https://github.com/PnX-SI/Nomenclature-api-module/blob/master/data/update1.3.1to1.3.2.sql La remarque vaut aussi pour l'ajout de champs dans t_datasets côté GeoNature.

Merci

FloVollmer commented 4 years ago

D'accord, donc dans le cas de geonature je dois modifier le fichier "2.2.1to2.3.0.sql" et pour Nomenclature-api-module le "1.2.1to1.3.2.sql", ou il faut que j'en crée un nouveau, du style "2.3.0to2.3.2.sql" et "1.3.2to1.3.3.sql" ?

FloVollmer commented 4 years ago

Comme je ne sais pas s'il y a une garantie sur l'ordre dans lequel les fichiers de migration sont exécutés je me suis dit que c'était mieux de tout mettre dans le même

camillemonchicourt commented 4 years ago

Oui maintenir les scripts de Migration est une partie un peu galère actuellement, même si des améliorations sont en cours (https://github.com/PnX-SI/GeoNature/issues/880). Le module Nomenclatures doit rester autonome et indépendant et peut être utilisé sans GeoNature. Donc il ne doit pas faire référence à GeoNature. La partie GeoNature (gn_meta) doit donc être faite dans le SQL de migration de GeoNature (https://github.com/PnX-SI/GeoNature/blob/develop/data/migrations/2.3.2to2.3.3.sql) et pas du sous-module.

Bien vu pour l'ordre, cela devra être indiqué dans les notes de la prochaine version de GeoNature. Comme par exemple dans les notes de version de la 2.0.1 : https://github.com/PnX-SI/GeoNature/blob/develop/docs/CHANGELOG.rst#201-2019-03-18 L'idéal est de le noter dès maintenant dans votre commit dans les notes de version de la prochaine version de GeoNature pour pas qu'on oublie d'ici là.

Merci.

FloVollmer commented 4 years ago

Ah oui d'accord. Ok je vais faire ça. Merci