Add a select field organizer on TouristicEvent model

[LePetitTim]

L'objectif de cette PR est de modifier le champ "Organisateur" sur les évènements touristiques.

Cette demande, issue initialement d'un besoin émis par le PNR des Volcans d'Auvergne, a pour objectif de permettre d'identifier par exemple les évènements organisés par des partenaires du Parc ou non.

Cela devra s'accompagner d'un développement dans GTR3 pour permettre aux utilisateurs d'avoir un filtre sur ce nouveau champ.

Actions à réaliser :

Côté Geotrek-Admin

• Ajout d'un modèle Organizer dans le module Tourisme de Geotrek, permettant de gérer des objets
• Modification du modèle Evenement Touristique pour transformer le champ "Organisateur" qui est actuellement un texte sous forme de clé étrangère vers le nouveau modèle.
  -> Concrètement cela permettra de gérer dans l'Admin de Geotrek la liste des organisateurs et dans la fiche détail d'un évènement de pouvoir sélectionner l'organisateur souhaité.
• Réalisation du script de migration de données pour assurer qu'aucune perte d'information n'aura lieu lors de la modification des modèles en base de données.
• Développement de tests unitaires pour assurer le fonctionnement de la fonctionnalité et éviter les régressions futures.
• Ajout d'un filtre dans l'API V2
• Adaptation des informations servies par l'API
• Modification des parsers du client pour que les évènements soient bien rattachés aux évènements touristiques (actuellement créés comme des contenus touristiques).

Côté Geotrek-Rando

• Adaptation des requêtes pour récupérer dans l'API le bon élément et vérifier qu'aucune régression n'a lieu lors de l'affichage de l'information dans la fiche détail d'un évènement
• Ajout de la possibilité d'utiliser le filtre dans l'API
• Modification de l'interface pour ajouter le filtre dans l'écran correspondant, modification du fichier de configuration associé pour permettre aux utilisateurs d'activer ou non ce filtre. Le filtre sera désactivé par défaut.

[cypress]

Passing run #6920 ↗︎

0

24

0

0

0

Details:

Merge 5a5a2c9 into dd07ee6...
Project: Geotrek-admin	Commit: d1f20fc99b ℹ️
Status: Passed	Duration: 04:32 💡
Started: Aug 29, 2023 10:24 AM	Ended: Aug 29, 2023 10:28 AM

_{This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.}

[camillemonchicourt]

En lien avec #3587 ?
Et notamment le point :

Organisateur : Basculer en liste (permet aussi de dissocier les animations parc ou non) - Avec picto pour l'afficher sur les résultats. Et en filtre. Pose la question de la redondance avec les contacts.

Si oui, il ne s'agit pas vraiment d'ajouter les organisateurs des évènements, mais plutôt de les basculer de texte à liste.

Le champs était en 256 et je pense pas qu'il faille le passer en 128.
Il y a une migration des données à faire pour tous les évènements existants qui ont déjà un organisateur en texte.
Et on avait parlé d'avoir un picto/logo optionnel pour chaque organisateur.
Ajouter aussi les organisateurs dans les filtres des évènements.

[babastienne]

@LePetitTim PR finalisée ou en draft ?

[codecov]

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (dd07ee6) 98.29% compared to head (5a5a2c9) 98.29%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3625   +/-   ##
=======================================
  Coverage   98.29%   98.29%           
=======================================
  Files         292      292           
  Lines       21695    21720   +25     
=======================================
+ Hits        21325    21350   +25     
  Misses        370      370           

Files Changed	Coverage Δ
geotrek/api/v2/filters.py	100.00% <100.00%> (ø)
geotrek/api/v2/serializers.py	99.89% <100.00%> (+<0.01%)	⬆️
geotrek/api/v2/urls.py	100.00% <100.00%> (ø)
geotrek/api/v2/views/__init__.py	100.00% <100.00%> (ø)
geotrek/api/v2/views/tourism.py	100.00% <100.00%> (ø)
geotrek/tourism/admin.py	100.00% <100.00%> (ø)
geotrek/tourism/models.py	99.72% <100.00%> (+<0.01%)	⬆️
geotrek/tourism/parsers.py	98.58% <100.00%> (+<0.01%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[LePetitTim]

Côté Geotrek-admin :

• mise en place des listes dans l'admin.
• recuperation des données depuis apidae tourinsoft et création d'un nouveau Organizer

L'api affiche toujours les organisateur de manière textuelle, comme avant. Il ne faut pas appeler une nouvelle route pour acceder aux organisateurs.

Il est dorénavant possible de filtrer sur les organisateurs d'événements en fonction du label de l'organisateur. Le 'lookup' utilisé est le contains :
la requete faite pour filtrer est donc organizer__label__contains ....

[babastienne]

Est-ce qu'il ne faudrait pas également ajouter une nouvelle route d'API permettant à GTR3 de récupérer directement toutes les valeurs du champ pour créer le filtre dans l'interface ?

[camillemonchicourt]

Oui, je pense que ce champs devrait fonctionner comme tous les autres et ne pas avoir un mode de fonctionnement, de recherche et de résultat différent.

[camillemonchicourt]

Pour cette info liée depuis une autre table en particulier, dans la route des événements touristiques (/touristicevent), on renvoie la valeur décodée (exemple "organizer": "Parc national des Cévennes",) et non pas son ID.
Je comprends l'intérêt et comme évoqué plus haut, c'est un sujet discuté plus globalement sur les réponses de l'API v2 pour laquelle il faut faire en permanence des "jointures".
Aussi pour le rétrocompatibilité, notamment de Geotrek-rando-v3.

Mais cela apporte aussi des divergences entre les différentes propriétés de l'API.
A minima, pour garantir la rétro-compatibilité mais avec plus de cohérence, je garderait la propriété organizer qui renvoie l'info à plat décodée, mais j'ajouterai aussi une propriété organizer_id sur la route /touristicevent.

[babastienne]

Oui, bien vu, je suis d'accord. A tracer dans un ticket pour qu'on puisse le rajouter à l'occasion.

Edit : c'est déjà fait, trop rapide @camillemonchicourt 😉

[Chatewgne]

Il n'y a pas de filtre par organisateur dans la vue liste des évènements, à faire ?