Skip to content
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

compatibilité Monitoring --> TaxHub --> GeoNature #340

Closed
gildeluermoz opened this issue Apr 25, 2024 · 4 comments
Closed

compatibilité Monitoring --> TaxHub --> GeoNature #340

gildeluermoz opened this issue Apr 25, 2024 · 4 comments

Comments

@gildeluermoz
Copy link

Version GN 2.13.4 taxhub 1.12.1 monitoring 0.7.1
Si code_liste n'est pas = à id_liste lors de la création d'une liste dans taxhub le module monitoring renvoie une erreur 400 BAD REQUEST

La route consultée lors de la recherche d'un taxon dans la liste des taxons observés par le module est la suivante :

retour json :
{"success": false, "message": "Code liste '102' inexistant"}

Ce soucis semble corrigé dans la 1.13.1 de taxhub qui nécessite GN 2.14. Il faut donc migrer vers GN2.14 + TH 1.14 + monitoring 0.7.2 ou corriger ses code_liste si on reste en GN 2.13

@camillemonchicourt camillemonchicourt changed the title compatibilité Monitoring --> TaxHbub --> GeoNature compatibilité Monitoring --> TaxHub --> GeoNature Apr 25, 2024
@camillemonchicourt
Copy link
Member

OK, on a un peu fait des allers et retours sur le sujet.

  • Au départ cette route interrogeait des id_liste
  • Puis on a ajouté des code_liste dans TaxHub en proposant de les privilégier
  • Pour la rétrocompatibilité on a alors renseigné tous les code_liste avec les id_liste existant
  • Mais cela a posé plusieurs soucis et on est revenu en arrière dans la 1.13.1 de TaxHub en effet. C'est détaillé ici : /allnamebylist/ attend un code_liste mais Occtax passe un id_liste... TaxHub#346

Mais il est possible qu'il faille répercuter cela dans Monitoring qui était peut-être passé sur des code_liste, mais je ne vois pas où, à vérifier.
Je vois des histoires d'id_liste ici par contre, mais je n'ai pas tout en tête : https://github.com/PnX-SI/gn_module_monitoring/blob/main/backend/gn_module_monitoring/config/generic/module.json#L78

@gildeluermoz
Copy link
Author

non je pense que monitoring est ok et qu'il interroge bien sur id_liste comme tu peux le voir dans l'url utilisée.
https://mondomaine.fr/taxhub/api/taxref/allnamebylist/102?search_name=coule&limit=20
C'est taxhub 1.12.1 qui considère que 102 est un code_liste. Et comme ma liste avait un code_liste = REP, taxhub ne trouve pas la liste 102 puisqu'il cherche une liste avec le code_lite=102
J'ai remis 102 dans code_liste et ça fonctionne.

@camillemonchicourt
Copy link
Member

Ahhh oui OK j'avais pas bien lu.

OK oui, c'est justement pour ça qu'on est revenu à des id_liste pour cette route. 👍

@camillemonchicourt
Copy link
Member

Corrigé avec TaxHub 1.13.1 en effet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants