Factoriser les pages détails

GeotrekCE / Geotrek-rando-v3

Geotrek public data website (version 3)

https://geotrek-rando-v3.readthedocs.io/stable/

MIT License

16 stars 14 forks source link

Factoriser les pages détails #510

Open camillemonchicourt opened 3 years ago

camillemonchicourt commented 3 years ago

Initialement les pages de contenus ont été dupliquées. On en a une pour les fiches rando, une pour les fiches outdoor, une pour les fiches services et une pour les fiches événements.

Alors que ces pages ont de nombreux éléments communs. Leur duplication entraîne des soucis d'éléments corrigés sur une page mais pas sur les autres, complique la maintenance, etc...

Une partie des pages a commencé à être factorisée, mais pas leur structure.

camillemonchicourt commented 2 years ago

A voir si c'est en lien, mais il serait aussi bien d'homogénéiser et de clarifier la manière de nommer et d'appeler les différents types de contenu dans le code.

Actuellement on a :

  practices: colors?.categories?.trek || '#001B84',
  outdoorPractice: colors?.categories?.events || '#E69736',
  categories: colors?.categories?.service || '#3B89A2',
  event: colors?.categories?.service || '#62AB41',

On devrait parler de :

trekPractices
outdoorPractices
serviceCategories
eventCategories

En lien aussi avec https://github.com/GeotrekCE/Geotrek-rando-v3/issues/507

Lilya-NS commented 2 years ago

Hello,

Qu'est-ce que tu entends par "factoriser" les pages ? Est-ce qu'on peut retrouver ce qui a déjà été fait ? Quels éléments ont besoin d'être mutualisés ?

Merci

camillemonchicourt commented 2 years ago

Actuellement, on a 4 types d'objets qui ont chacun leur page détail (rando, outdoor, service, événements). Ils ont chacun une structure de données différente, mais aussi beaucoup d'éléments et de fonctionnalités qui sont communes. Hors, une grande partie de ces éléments communs sont dupliqués dans chaque type de page détail au lieu d'être factorisés. Il faudrait donc faire une analyse pour voir ce qui est factorisable ou non. Mais cela nécessite peut-être une première analyse globale des différences et points communs entre les fiches détail des différents objets et plus de recul sur le sujet ?

camillemonchicourt commented 1 year ago

Un travail de factorisation partielle des pages détails a été réalisé dans la PR https://github.com/GeotrekCE/Geotrek-rando-v3/pull/934 pour ne plus dupliquer le code commun à chaque page détail des randos, sites et parcours outdoor, contenus et événements touristiques.

Celle-ci inclut aussi des nouvelles fonctionnalités avec l'ajout du fichier de configuration details.json :

Possibilité de masquer certaines sections des pages détails
Possibilité d'afficher ou non les sections dans les ancres des pages détails
Possibilité de réordonner les sections dans les pages détails

En complément, du ménage a été fait sur les pages détail, comme le fait de ne plus avoir 2 instances chargées de la carte mais une seule - https://github.com/GeotrekCE/Geotrek-rando-v3/pull/939

dtrucs commented 1 year ago

Depuis l'ajout des sections dynamiques, il reste une dernière phrase de refacto:

Renomer le module details en trek (une partie du vocabulaire a déjà été fait)
Revoir tous les adapter des modules trek, touristicContent, touristicEvent, outdoorSite, OutdourCourse et homogénéiser les valeurs des props via une une nouvelle interface (au sens typescript du terme) Details qui va documenter la construction de la page selon son type de contenu.
Reprendre le composant details.tsx qui prend toutes ces valeurs et supprimer les autres pages "details" (touristicContentUI.tsx, touristicEventUI.tsx, outdoorSiteUI.tsx, OutdourCourseUI.tsx)