-
Notifications
You must be signed in to change notification settings - Fork 103
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
Erreur sur l'API taxhub/api/taxref/allnamebylist/<idliste> avec GeoNature 2.12.3 et TaxHub 1.12.0 #2636
Comments
Effectivement, il y a un couac entre la version de TaxHub installé dans le venv de Geonature (qui fournit les migrations à appliquer) et la véritable version de TaxHub installée.. |
Solution équivalente testée qui fonctionne aussi : mettre la bonne version de taxhub dans le requirements.txt de GeoNature avant de lancer le migration.sh. |
Pour éviter un conflit de dépendances, il faut aussi mettre à jour la version de
|
Oui c'est "normal", depuis le passage à Alembic, c'est GeoNature qui se charge de mettre à jour toutes les branches dont celles des schémas Quand on sort une nouvelle version de TaxHub qui contient des évolutions du modèle de données, il faut aussi sortir une version de GeoNature qui fait référence au niveau de ses dépendances à cette nouvelle version de TaxHub. Idem que PnX-SI/TaxHub#396 On a en effet sorti TaxHub la version 1.12.0 de TaxHub, en prévision de la version 2.13.0 de GeoNature, mais on a un peu temporiser celle-ci. C'est donc fait dans la branche DEVELOP - f6f00e9 En attendant, il ne faut pas mettre à jour TaxHub si on utilise GeoNature. Ou alors on aurait pu sortir 2.12.4 de GeoNature juste pour ça, mais c'est dommage. Ce qu'on peut faire, c'est indiquer dans les notes de version de TaxHub 1.11.2, 1.11.2 et 1.12.0 : "Compatible avec GeoNature 2.13.0 ou plus" (même si cette version n'existe pas encore...). Avec la v2 de TaxHub en cours de développement, le lien entre TaxHub et GeoNature sera plus assumé et packagé, donc on devrait avoir moins de soucis de ce type. |
TaxHub 1.12.0 intégré avec la version 2.13.0 de GeoNature. |
Version
GeoNature 2.12.3 avec TaxHub 1.12.0
Description du bug
Après mise à jour des 3 applications UsersHub, TaxHub et GeoNature sur leurs dernières releases, une erreur est soulevée dans OccTax sur la route
https://[...]/taxhub/api/taxref/allnamebylist/<idliste>
. Ceci est dû aux différences de version de TaxHub entre la version installée (v1.12.0) et la dépendances installée dans le venv de geonature (v1.11.1). Par conséquent, des migrations sont donc manquantes dans ledb autoupgrade
de geonature.Solution, installer taxhub 1.12.0 dans le venv geonature (ou installer TaxHub 1.11.1)
source geonature/backend/venv/bin/activate pip install taxhub==1.12.0
Comportement attendue
Consultation de l'API TaxHub sans erreur ;)
Comment reproduire
Installer TaxHub 1.12.0 avec GeoNature 2.12.3
Logs
Extrait du fichier
/var/log/taxhub.log
.The text was updated successfully, but these errors were encountered: