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

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

Closed
wants to merge 1 commit into from

Conversation

peblair
Copy link

@peblair peblair commented May 13, 2021

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
Copy link
Collaborator

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
Copy link
Author

peblair commented May 14, 2021

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
Copy link
Collaborator

nhumblot commented May 15, 2021

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

Hi! 👋

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

And your french is very good! 👍 🇫🇷

@peblair
Copy link
Author

peblair commented May 16, 2021

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 :

  • Ceux qui cherchent d'un créneau proche de tout type, Chronodose ou pas
  • Ceux qui cherchent une Chronodose et veulent bien aller dans un autre département dans la même région (bien sûr, ciblant principalement ceux habitent en Île-de-France, puisque je n'imagine pas que quelqu'un à Bayonne veuille aller à La Rochelle)

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
Copy link
Collaborator

fcamblor commented Aug 5, 2021

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

@fcamblor fcamblor closed this Aug 5, 2021
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

Successfully merging this pull request may close these issues.

3 participants