CovidTrackerFr / vitemadose-front

Interface utilisateur de l'outil ViteMaDose
https://vitemadose.covidtracker.fr
Other
80 stars 51 forks source link

Ajouter la possibilité de rechercher par région #201

Closed peblair closed 3 years ago

peblair commented 3 years ago

Bonjour ! En tant qu'utilisateur du site, j'ai senti que la possibilité de rechercher par région manquait. Par exemple, j'étais intéressé de connaître les disponibilités des Chronodoses dans toute l'Île-de-France, mais il n'existe pas de bonne méthode pour faire cela actuellement. Je vous propose donc cette addition au site, qui permet aux utilisateurs de filtrer les centres par région.

Voici quelques captures d'écran:

Screen Shot 2021-05-13 at 11 17 57 AM Screen Shot 2021-05-13 at 11 21 17 AM
fcamblor commented 3 years ago

Bonjour, et merci pour votre contribution :-)

Je suis partagé sur la valeur ajouté de cette fonctionnalité car on a déjà énormément de centres (350+) lorsqu'on fait une recherche sur Paris (75) qui inclut les départements limitrophes (92, 93 et 94)

Si on rajoutait tous les départements de la région île de france (77, 78, 91 et 95) on arriverait à 495+ centres.

Pour des raisons de performances sur certains navigateurs (safari pour ne pas le nommer), on a dû limiter à 180 centres le nombre de centres affichés sur la page de résultats. D'autre part, il existe un filtre implicite à 100km (qui risque de beaucoup limiter l'aspect recherche par région)

Je me laisse un peu de temps pour réfléchir (et peut-être à d'autres personnes pour réagir) sur les points évoqués ci-dessus.

peblair commented 3 years ago

Je comprends totalement vos inquiétudes, car c'est vraiment un grand nombre de résultats. Ceci étant dit, je pense qu'une bonne solution serait de combiner (d'une manière ou d'une autre) les changements ici avec ceux qui sont proposés en #143 . Je n'ai pas vérifié, mais j'imagine que le plupart de ces 350+ centres dans Paris sont des pharmacies; parce que ces lieux apportent seulement le vaccin AstraZeneca (et peut-être Janssen ? Je suis pas sûr...mais je m'égare), le public ciblé pour la fonctionnalité Chronodose (mois que 50 ans) n'est probablement pas intéressés par eux, non ? Si c'est le cas, cette fonctionnalité en combinaison avec la capacité de filtrer les centres selon quels types de vaccins sont disponibles serait très utile tout en restant performant. Je ne sais pas si vous êtes d'accord.

Aussi, peut-être on peut paginer les résultats? Ou cela ne résoudrait pas le problème ?

(Pardon mon français simple, ce n'est pas ma première langue)

nhumblot commented 3 years ago

Aussi, peut-être on peut paginer les résultats? Ou cela ne résoudrait pas le problème ?

Hi! :wave:

For your information, an infinite scroll implementation is in progress with #37 :slightly_smiling_face:

And your french is very good! :+1: :fr:

peblair commented 3 years ago

Merci @nhumblot ! J'avoue que j'ai fait relire quelqu'un, mais je suis en train d'apprendre 🙂. Now I am a bit confused about whether to respond in English or French 😅. I'll stick with French, just because I think that is better for most of the project's contributors (and it's good practice for me) 🇫🇷.

Sur la base de ma lecture de #37 (bien que je ne sois pas sûr que c'est 100% correct), il me semble que le plan est de laisser le modèle de données tel quel (un gros fichier récupérant par le serveur VMD), et faire un infinite scroll sur le client. Si c'est le cas, je pense qu'une combinaison du infinite scroll, #143, et ça seront très utile pour les utilisateurs sans sacrifier le facilité d'utilisation ou la performance de la page (noter que les modifications proposés ici n'ont pas d'impact sur la quantité de données que nous gardons, mais plutôt le nombre de centres qui sont montrés aux utilisateurs).

Je pense que la combinaison de ces fonctionnalités permettraient au site de supporter bien les profils d'utilisateurs suivants :

Après reflexion, une autre amélioration pour le fonctionnement de ce PR (encore, principalement en pensant à des utilisateurs en Île-de-France) serait si on triait les résultats par la distance au centre de la région. Je serai content de faire ce changement, mais on devrait d'abord décider si on veut intégrer ce PR ou pas.

fcamblor commented 3 years ago

Hello ici ✋

Il n'y a pas eu beaucoup d'activité sur la PR, et je pense que la dernière MEP introduisant la possibilité de customiser le filtre de distance (et affichant les jours de disponibilité) devrait fixer le problème évoqué dans cette PR

=> Je ferme la PR

@peblair n'hésites pas à te manifester si tu souhaites persévérer et que je réouvre la PR