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

Modifications de la route /taxref pour ajouter medias et attributs #498

Open
andriacap opened this issue May 23, 2024 · 2 comments
Open

Comments

@andriacap
Copy link

Bonjour,

Je créé ce ticket au sujet des discussions qui ont été entamées sur le dépôt de GeoNature citizen en vu de modifier la route /taxref . Pour résumer, cette modification suggérerait de pouvoir passer en params de types "medias" et "attributs" pour pouvoir retourner les médias et attributs associés à une ou plusieurs espèces ( à discuter sur las yntaxe des params).

Ces modifications font références au fait que dans Citizen pour le moment , il y a un appel successif pour plusieurs routes coté backend pour charger au préalable la liste des espèces d'un programme .

Cette modification permettrait d'éviter d'avoir à faire un appel à une route pour avoir dans un premier temps toutes les informations de la liste des espèces (cd_nom, nom scientifique etc) puis sur la base de cette liste appelé autant de fois la route qui sert à récupérer les médias pour récupérer chaque médias de chaque espèce.

Je laisse @camillemonchicourt et @hypsug0 me corriger, compléter au cas où il manquerait des informations.

Référence isssue dans GeoNature Citizen : PnX-SI/GeoNature-citizen#327
Référence PR en draft dans GeoNature Citizen : PnX-SI/GeoNature-citizen#381

Merci d'avance

@camillemonchicourt
Copy link
Member

OK, en effet, il nous semble OK et pertinent d'enrichir la route /taxref/ qui serait la route de base pour récupérer les taxons et leurs éventuels infos associées.
Par contre, il faut voir comment faire ne sorte que ces objets enfants (médias et attributs) ne soient pas forcément renvoyés de base, mais seulement si on les demande, pour ne pas alourdir la réponse de la route par défaut et pour son usage actuel.
Mais c'est déjà certainement le cas pour les listes de taxons qui ont été ajoutées à la route /taxref/ dans la version 1.13.1. Donc voir comment c'est fait et faire pareil, il me semble.

@andriacap
Copy link
Author

Salut Camille,

Oui il s'agit bien de paramètres optionnels à rajouter pour ne pas impacter l'existant

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

2 participants