search

CovidTrackerFr / vitemadose-front

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

Reverting jawg results map #237

Closed fcamblor closed 3 years ago

fcamblor commented 3 years ago

Cette Pull Request est

Description

Cette PR est un retour arrière sur #228 (et par extension #236) proposé par @Joxit, et qui suscite pas mal de débats sur la manière dont intégrer une carte dans l'UI de la page résultats.

Pour moi, ce changement n'a pas été suffisamment discuté, et je me sens mal à l'aise de bloquer toute MEP basée sur dev en attendant de trouver une issue à cette problématique, d'où ce rollback.

Dans une autre mesure, l'onglet carte me semble incompatible avec l'orientation qu'on est parti pour prendre sur l'éligibilité des 18+, basé sur un bandeau de dates de disponibilités (cf #233), et qui devrait sortir assez rapidement.

L'idée n'est pas de disqualifier la fonctionnalité, mais de se laisser davantage de temps pour discuter à son intégration (ou non) via peut-être des changements dans l'implémentation initiale.

Les points que je souhaiterais voir discutés plus avant dans une PR ré-intégrant ces commits seraient :

Je vais rouvrir, dans la foulée, une autre PR qui ré-intègre les commits revertés ici, afin d'initier la discussion sur les points évoqués ci-dessus.

fcamblor commented 3 years ago

La PR ré-intégrant ces changements (et initiant la discussion sur le sujet) est là : #238

fcamblor commented 3 years ago

Merci à tout le monde pour votre réactivité :)

loicortola commented 3 years ago

Hello @fcamblor bien noté pour cela. Pour infos, on a échangé au préalable avec @rozierguillaume sur ce sujet, on aurait surement dû plus intéragir avec la communauté active là dessus. N'étant pas au courant de tous vos process on s'est dit qu'il valait peut-être mieux lancer quelque chose, et discuter ensuite ^^.

Que pensez-vous de faire un point en visio avec les parties prenantes pour qu'on comprenne bien vos envies ?

Pour le côté remplacement OSM / Jawg, je comprends ta réaction mais je ne suis pas vraiment d'accord : la communauté OSM n'a pas pour vocation de mettre à disposition des fonds de plan pour être utilisés à cette échelle mais plutôt pour que les contributeurs suivent leurs contributions. Donc grosso-modo, vous utilisez des fonds de plans qui ne sont pas faits pour ça (https://operations.osmfoundation.org/policies/tiles/). C'est pour ça d'ailleurs que le look & feel de la carte n'est pas vraiment adapté à une UI proche de ce que les gens ont l'habitude de voir niveau couleurs et densité d'infos.

On est ravis de contribuer à ce projet, on ne demande rien d'autre que de joindre nos petites mains aux votres pour rendre ce produit plus joli, et bien sur qu'on serait ravis si notre attribution est présente quelque part ☺️.

Si ça ne fait pas le job, et que ça ne vous convient pas, on est preneurs de savoir ce qui le serait, sauf si vous n'avez pas besoin de quoi que ce soit. On serait ravis de mieux l'intégrer à l'UX de votre future PR ;)

fcamblor commented 3 years ago

Pas de soucis @loicortola, je n'ai (vraiment !) rien contre JAWG, c'est juste que le merge de la PR arrivait à un moment un peu compliqué par rapport aux sujets qu'on souhaitait MEP (d'où mon retour arrière, mais j'ai créé #238 pour qu'on reparle de tout ça à tête reposée)

J'avais bien en tête les restrictions d'usage d'OSM, mais compte tenu de notre usage actuel (la page centres complètement à coté du scénario nominal de VMD qui est la recherche + résultats) je ne pense pas qu'on était trop borderline sur la charge que cela représentait (les analytics que j'ai me parlent de moins de 100 affichages de la page des centres sur tout le mois de Mai 😅)

Mais je te rejoins sur le fait que si on affiche la carte sur la page de résultats, ça risque de ne pas être la même histoire :-)

On parle de tout ça sur #238 ? :)