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

Appel à Taxhub directement depuis le front pour récupérer la liste des taxons #324

Closed
mvergez opened this issue Dec 7, 2021 · 2 comments

Comments

@mvergez
Copy link

mvergez commented Dec 7, 2021

Problématique

Comme évoqué dans #323, il serait plus judicieux de supprimer la route /taxonomy_lists/<id_liste_taxhub>/species/ du backend de citizen et de la remplacer par un appel Taxhub directement : /api/biblistes/taxons/<id_liste_taxhub>.

A première vue il manquerait uniquement le media_id à récupérer auprès de Taxhub (donc en appelant une autre route)

Proposition d'implémentation

Compléter le service Taxhub déjà présent pour éviter des routes en dur dans le code


J'essaie de faire une PR

@TheoLechemia
Copy link
Member

Salut,
Si c'est pour proposer une liste avec un typeahead (autocomplété), la route est plutôt celle-ci :
https://github.com/PnX-SI/TaxHub/blob/master/apptax/taxonomie/routestaxref.py#L341
Et on privilégie maintenant la recherche via le "code_liste", plutôt que "l'id_list" qui est un serial.

@mvergez
Copy link
Author

mvergez commented Dec 8, 2021

Salut Théo,
Merci beaucoup pour la route, je vais partir sur celle là.
Pour l'id_list je pense qu'il faudra changer un peu plus de choses dans Citizen.

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