GeotrekCE / Geotrek-admin

Paths management for National Parks and Tourism organizations
https://geotrek.fr
BSD 2-Clause "Simplified" License
133 stars 76 forks source link

APIv2 ne renvoie pas tous les bureaux d'information touristique dans le cas des itinérances #3508

Open Chatewgne opened 1 year ago

Chatewgne commented 1 year ago

La liste des bureaux d'information touristique (/api/v2/informationdesk/) ne renvoie pas les BIT qui sont uniquement associés au trek enfant (non-publié) d'une itinérance publiée.

Cela peut causer un crash de la page détail du trek enfant en question sur Rando-v3.

Piste : essayer de rajouter ce genre de query dans le filtre filter_queryset_related_objects_published dans le cas où related_name=='treks'

related_field_name = 'treks__trek_parents__parent__published_{}'.format(lang)
q &= Q(**{related_field_name: True})

Il va falloir faire attention à l'impact de l'ajout de cette query sur les performances de l'API

camillemonchicourt commented 1 year ago

Oui identifié ici - https://github.com/GeotrekCE/Geotrek-rando-v3/issues/718

Il faudrait surtout faire en sorte que les pages de Geotrek-rando-v3 ne crashent pas si un lieu de renseignement n'est pas renvoyé par l'API, c'est pas normal que cela soit le cas.

Je changerai rien côté API.