3liz / qgis-gestion_base_adresse-plugin

Extension QGIS de gestion d'une base adresse française
https://docs.3liz.org/qgis-gestion_base_adresse-plugin/
4 stars 4 forks source link

Create upgrade_to_0.8.0.sql #100

Closed Jersig closed 3 years ago

Jersig commented 3 years ago

Update of the latest production version of the "adresse" schema. It implements various objects type : • Functions • tables, sequences, default values • views • indexes • triggers • constrains • comments This upgrade aims to create new tables and setup several functions which automatically update different fields in our tables. It allow to produce assessment view used to export data with a structure expected by the project partners.

Gustry commented 3 years ago

Merci pour la contribution @Jersig Est-ce qu'il y a une raison pour vouloir faire 2 versions de l'extension 0.8.0 ici #100 et aussi une 0.9.0 dans #101 ? Pourquoi pas dans la même release ?

Pour faire du SQL, il y a 2 choses :

Il faut que les 2 soit synchro.

Je viens de d'approuver le lancement des tests et avec surprise, le test est vert ... Je pense qu'il y a un soucis de notre côté suite à la migration Travis vers GitHub récemment. Parce que ton PR n'ayant que un fichier de migration, il y a donc une différence si quelqu'un fait l'installation d'une nouvelle base. Je dois vérifier ca.

Gustry commented 3 years ago

@Jersig Je me suis permis de faire un "rebase", CAD mettre à jour ton PR, suite à la découverte d'un problème possible ce matin.

Donc comme dit ci-dessus, le test des migrations plante désormais car le PR ne couvre pas l'installation d'une base.

Merci pour la puce à l'oreille :)

rldhont commented 3 years ago

Ajouter BEGIN et COMMIT aux fichiers SQL

pdrillin commented 3 years ago

@Gustry @rldhont j'ai trouvé, vous pouvez review

rldhont commented 3 years ago

LGTM