Reverting jawg results map

[fcamblor]

Cette Pull Request est

• Un correctif
• Une nouvelle fonctionnalité
• Un retour arrière

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 :

• Le remplacement d'OSM par JAWG (ça fait un peu placement produit pour quelque chose qu'on n'avait pas particulièrement demandé .. j'aimerais que l'équipe discute de ce point avant de le valider, afin d'éviter tout malaise)
• La forme utilisée pour la carte : actuellement il s'agit d'un onglet (qui est incompatible avec Nouveau mode de fonctionnement en prévision de l'éligibilité 18+ : introduction du calendrier des créneaux #233), j'aimerais qu'on explore d'autres options (par exemple, un affichage en popup)
• Le bien fondé de la fonctionnalité : certains sont contre tout affichage de carte, d'autres sont pour l'affichage d'une carte par centre, et enfin la proposition ici est d'afficher tous les centres de la liste sur la carte (ce qui peut faire beaucoup dans les zones denses en lieux de vaccination comme Paris)

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]

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

[fcamblor]

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

[loicortola]

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]

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 ? :)