Nouveau mode de fonctionnement en prévision de l'éligibilité 18+ : introduction du calendrier des créneaux #233

fcamblor · 2021-05-25T11:55:24Z

Cette Pull Request est

Un correctif
Une nouvelle fonctionnalité

Description

L'idée de cette PR est de fournir plusieurs changements sur le formulaire de recherche VMD du fait de l'ouverture à un public bien plus large à la vaccination le 31 Mai.

Les changements majeurs effectués dans cette PR :

Affichage d'un bandeau affichant les créneaux disponibles par jour, et permettant de sélectionner un jour en particulier pour la prise de rdv (par défaut, le jour le plus proche avec des disponibilités est sélectionné)
Du fait de ce nouveau champ, suppression du critère de tri (lorsqu'on est en recherche par commune, il n'est plus opportun de laisser le choix à l'utilisateur de trier par date de première dispo puisqu'il a dorénavant le calendrier qui auto-sélectionne le premier jour avec des créneaux disponibles)
Affichage d'un filtre supplémentaire permettant de sélectionner un type de vaccin spécifique à la tranche 18-55 (vaccins à base d'ARNm) versus "tous les vaccins". Étant donné que le plus gros des utilisateurs à venir seront vraisemblablement dans la tranche 18-55, pré-selection de ce critère.
À noter : si aucun calendrier n'est fourni dans le JSON (c'est le cas pour la plupart des communes, sauf celles du 33 et du 17 ainsi que leur départements limitrophes), le bandeau des jours ainsi que le filtre sur le type de vaccin n'apparaîtront pas car il est alors impossible de connaître une répartition par jours des créneaux de type ARNm/Tous
Le bandeau jours + le critère type de vaccin ne sont évidemment pas affichés sur l'onglet chronodose, qui a une configuration spécifique pour ça (créneaux sur aujourd'hui+demain, et vaccins ARNs uniquement)
Un filtre permettant d'ajuster la distance de recherche a été mis à disposition, sur les 2 onglets recherche standard / chronodose

Fonctionnalités visibles ici : https://dev.vitemado.se/disponibilite-generale/

⚠️ Pour le moment, le back ne mettant pas à disposition les données, un mock de ces dernières (dans le format évoqué dans CovidTrackerFr/vitemadose#519) a été hardcodé sur cette branche.
Ça signifie que les données n'évolueront pas dans le temps, et cela explique aussi pourquoi vous verrez sûrement un calendrier avec des dates dans le passé (dans la vraie vie, ça ne se passera pas comme ça)

Fix #96

nhumblot

Bravo c'est vraiment bien. 👏 👍

Ça marche bien à l'usage. Deux trois ajustement au niveau graphique à terme. Peut-être retirer complètement la gestion chronodose à terme ?

J'ai ajouté un petit changement pour que ce soit fonctionnel sur localhost, j'avais des erreurs 404 sur mon poste. J'ai ajouté un petit check sur la méthode upperFirst au cas où on reçoive à terme des données bizarres (on a déjà eu le cas par le passé).

Félicitations en tout cas. 🙂

nhumblot · 2021-05-25T19:27:12Z

src/components/vmd-input-range-with-tooltip.component.ts

+        return this.options.length-1;
+    }
+    @internalProperty() get bubbleLeft() {
+        const percentageValue = Math.round(this.indexOptionSelectionnee*10000/this.max)/100;


Un détail (à négliger ?) car les options sont hardcodées dans vmd-rdv-view.ts mais si on venait à définir une seule option on aurait une division par zéro.

nhumblot · 2021-05-25T19:36:24Z

src/state/State.ts

+            if(daySelectorDisponible) {
+                return creneauxPourLieu?creneauxPourLieu.creneaux.filter(cbt => cbt.tags.includes('preco18_55')).length:0;
+            }
+            throw new Error("We're not supposed to call cardAppointmentsExtractor() on 18_55 without day selector !")


Ça me semble bien car je n'ai pas vu de situation où je pouvais être en 18_55 sans sélectionner une journée. Mais j'ai l'impression que c'est également le cas en mode standard, est-ce qu'on ne devrait pas avoir une logique similaire aussi pour 'standard' ? J'ai l'impression qu'on n'a pas de situation où on n'a pas sélectionné de date. 🤔

fcamblor · 2021-05-26T00:39:25Z

J'ai des problèmes sur mobile où le slider ne s'affiche / ne fonctionne pas :(

EDIT: fixé dans c5a968f

francoisBouchet · 2021-05-26T12:03:58Z

hier soir je m'étais amusé à faire les blocs jours, si des fois il y a des trucs qui peuvent t'intéresser pour "raffiner" Frédéric, enjoy.

html Day2Day-Francois.zip

fcamblor · 2021-05-26T23:01:31Z

@francoisBouchet j'ai fait quelques modifs ce soir pour intégrer tes remarques remontées dans mattermost : cf 1ad3b85 4860c7b 79775d0

Ça donne ça :

…for department search) or 'distance' (for commune search)

… of directly in vmd-rdv

…chTypeConfig map

fcamblor · 2021-06-03T22:54:27Z

On utilise le même code couleur et visuel pour deux blocs qui ne se comportent pas de la même manière : le bouton prendre rendez-vous, et le jour sélectionné dans le calendrier).

@francoisBouchet @Luwangel
Que dites-vous de :

Peut-être un poil trop pâle non ?

(en effet, avoir une couleur différente du mauve, mais sans opacité, serait sûrement mieux ... des idées ?)

Luwangel · 2021-06-07T11:23:49Z

On utilise le même code couleur et visuel pour deux blocs qui ne se comportent pas de la même manière : le bouton prendre rendez-vous, et le jour sélectionné dans le calendrier).

@francoisBouchet @Luwangel
Que dites-vous de :

Peut-être un poil trop pâle non ?

(en effet, avoir une couleur différente du mauve, mais sans opacité, serait sûrement mieux ... des idées ?)

En fait tu n'as pas besoin d'indiquer en couleur le jour courant, puisqu'il s'agit par défaut du jour le plus à gauche libre. La même interface que ce que tu proposes sans le mauve autours du vendredi me semble parfaite.

Sur airbnb, les jours non réservables sont de la même couleur que les jours dans le passé. Et ça me semble assez clair finalement.

fcamblor · 2021-06-07T12:25:09Z

Le jour en mauve foncé c'est pas le jour courant, c'est le jour sélectionné :-)

Légende :

gris = 0 créneaux
mauve foncé = jour sélectionné
blanc = jour sélectionnable
blanc avec contour mauve clair = hover sur jour sélectionnable

bilelz · 2021-06-07T14:08:55Z

src/components/vmd-upcoming-days-selector.component.ts

+                selected: this.dateSelectionnee === cq.date, 
+                selectable: this.isSelectable(cq),
+                empty: this.dateSelectionnee !== cq.date && appointmentCount === 0
+              })}" @click="${() => this.jourSelectionne(cq)}">


hello, pour un peu plus d'accessibilité (testé avec TalkBack - chrome/android)

Suggested change

})}" @click="${() => this.jourSelectionne(cq)}">

})}" @click="${() => this.jourSelectionne(cq)}"

@keydown="${(e:KeyboardEvent) => e.key === 'Enter' && this.jourSelectionne(cq)}"

role="button" tabindex="0">

Ceci n'est qu'une recommendation, pas une obligation. Sinon, reformuler différemment. Source: https://www.service-public.fr/particuliers/actualites/A14722

- Il faut avoir + de 12 ans - Suppression du reste, qui était redondant face au 1er point

- ajout de la balise sémantique time pour l'accessibilité - le bloc est maintenant plus compact - correction d'une erreur dans l'URL affichée

Je n'ai pas vraiment d'avis tranché sur l'orthographe de ce mot, je corrige juste l'incohérence : partout ailleurs on met un tiret. (pas vraiment en lien avec la PR, mais j'en profite)

Merge on main

mccob · 2021-07-12T20:07:28Z

Bonsoir,

Cette PR est-elle toujours prévue ?

fcamblor · 2021-07-13T07:58:57Z

Oui, mais elle nécessite pas mal d'alignements (prise en compte du nouveau fichier sur tous les scrappers de toutes les plateformes) sur le back (et bloque pas mal les développements sur le front en attendant, vu les impacts que cela a)

Mise à jour conditions éligibilité vaccin

src/components/vmd-upcoming-days-selector.component.scss

Floby

C'est top ! je suis content de voir que ça fonctionne avec toutes les plateforme que Sacha a rajouté =)

Ça donne plein d'idées de trucs qui pourraient être un chouïa mieux, mais moi je dis ça ship. Les 2 trucs qui vont avoir de l'importance rapidement pour moi c'est de s'assurer qu'une description accessible du jour selectionné est disponible (mais je laisse @bilelz dire comment parce que je suis nul) et pouvoir arriver directement sur un jour selectionné via la navigation mais evidemment il y a plein de cas aux limites.

src/components/vmd-upcoming-days-selector.component.scss

Co-authored-by: Florent Jaby <[email protected]>

fcamblor · 2021-07-21T17:16:42Z

Merci @Floby pour les propositions :)

…ments in upcoming-days-selector component

…nes are Pfizer (AZ & J are most likely not used)

fcamblor · 2021-07-24T11:12:45Z

Je viens de pusher une nouvelle version de la feature :

Suppression du critère de recherche sur le type de vaccin (car aujourd'hui, plus aucune injection d'AZ ne semble faite, et J&J reste extrêmement minoritaire dans son utilisation)
J'ai fait en sorte de "fusionner" les jours sans créneau dispos dans le calendrier dès lors :
- Qu'on a au moins 3j consécutifés sans créneau dispo
- Le premier jour restera systématiquement affiché
- Les jours suivants seront remplacés par une entrée "Jours sans créneaux", cliquable (si cliquée, les jours sans créneau sont alors affichés)

Une fois cliqué :

fcamblor requested review from Floby, Luwangel, bilelz and nhumblot May 25, 2021 11:55

nhumblot reviewed May 25, 2021

View reviewed changes

fcamblor force-pushed the disponibilite-generale branch from 1463fdd to ebb38a7 Compare May 26, 2021 00:14

fcamblor force-pushed the disponibilite-generale branch from ebb38a7 to ecc4a10 Compare May 26, 2021 00:49

fcamblor mentioned this pull request May 26, 2021

Ajout d'un onglet 'Carte' avec tous les RDV d'un département ou prés d'une ville #228

Merged

5 tasks

This was referenced May 27, 2021

Reverting jawg results map #237

Merged

Map on search results #238

Open

nhumblot mentioned this pull request May 27, 2021

Ajout de la fonctionnalité permettant de choisir le type de vaccin #143

Open

fcamblor force-pushed the disponibilite-generale branch 2 times, most recently from c04dc98 to 3717486 Compare May 27, 2021 22:30

fcamblor added 14 commits May 30, 2021 22:46

Merge remote-tracking branch 'origin/dev'

e405b96

introduced search by type vaccin

609a027

introduced upcoming day selector

e54cfcb

getting rid of critereTri property, this is now hardcoded to 'date' (…

2564f02

…for department search) or 'distance' (for commune search)

introduced distance search field

39be1cc

introduced search by horaires criteria

762e851

handling multiple departments complexity in State.lieuxPour() instead…

17de5b3

… of directly in vmd-rdv

handling creneaux quotidiens in centres json file

e4d7de4

using dedicated component for upcoming days selector

20e5ad8

extracted search type specificities into a dedicated/centralized Sear…

8fb4677

…chTypeConfig map

extracted lieuxDisponibles into afficherLieuxParDepartement() impl

8e3ba86

introduced day-level statistics through rdvsDuJourSelectionne property

a37a4df

hiding type vaccin on chronodose tab

bc6ff17

hiding horaires rdv criteria

4b7f684

fix error when creneauxQuotidienSelectionnes could be null

99c5331

bilelz reviewed Jun 7, 2021

View reviewed changes

fcamblor mentioned this pull request Jun 11, 2021

[proposition d'évolution] ajuster la granularité de certaines statistiques #247

Open

Joxit mentioned this pull request Jun 14, 2021

Map on search results #248

Draft

2 tasks

a2br and others added 5 commits June 21, 2021 08:48

Suppression mention femmes enceintes

f404fd1

Ceci n'est qu'une recommendation, pas une obligation. Sinon, reformuler différemment. Source: https://www.service-public.fr/particuliers/actualites/A14722

Mise à jour des informations sur l'éligibilité à la vaccination

973bd07

- Il faut avoir + de 12 ans - Suppression du reste, qui était redondant face au 1er point

Reformulation pour les mineurs + modifications mise en page.

a5a28a8

- ajout de la balise sémantique time pour l'accessibilité - le bloc est maintenant plus compact - correction d'une erreur dans l'URL affichée

Correction nom Covid19 -> Covid-19

4c07881

Je n'ai pas vraiment d'avis tranché sur l'orthographe de ce mot, je corrige juste l'incohérence : partout ailleurs on met un tiret. (pas vraiment en lien avec la PR, mais j'en profite)

Merge pull request #252 from CovidTrackerFr/dev

6e7c5df

Merge on main

JulesMichael mentioned this pull request Jul 13, 2021

Filtres de recherche CovidTrackerFr/vitemadose#594

Closed

fcamblor added 3 commits July 17, 2021 11:14

Merge pull request #251 from a2br/infos-vaccin

7291243

Mise à jour conditions éligibilité vaccin

using production url instead of staging one

cb58b60

Merge remote-tracking branch 'origin/main' into disponibilite-generale

675c263

Floby reviewed Jul 20, 2021

View reviewed changes

src/components/vmd-upcoming-days-selector.component.scss Outdated Show resolved Hide resolved

Floby reviewed Jul 20, 2021

View reviewed changes

src/components/vmd-upcoming-days-selector.component.scss Show resolved Hide resolved

fcamblor changed the title ~~Nouveau mode de fonctionnement en prévision de l'éligibilité 18+~~ Nouveau mode de fonctionnement en prévision de l'éligibilité 18+ : introduction du calendrier des créneaux Jul 21, 2021

fcamblor and others added 2 commits July 21, 2021 19:12

better css for overflow-x

9b76b9c

Co-authored-by: Florent Jaby <[email protected]>

highlighting today in the calendar

0fadb3e

Co-authored-by: Florent Jaby <[email protected]>

fcamblor added 3 commits July 21, 2021 19:22

minor cosmetic fixes

9b35a8e

introduced single entry for consecutive days with 0 available appoint…

7815813

…ments in upcoming-days-selector component

getting rid of vaccine type search filter, now that most of our vacci…

4e0d837

…nes are Pfizer (AZ & J are most likely not used)

fcamblor merged commit d2f46e4 into dev Jul 25, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Nouveau mode de fonctionnement en prévision de l'éligibilité 18+ : introduction du calendrier des créneaux #233

Nouveau mode de fonctionnement en prévision de l'éligibilité 18+ : introduction du calendrier des créneaux #233

fcamblor commented May 25, 2021 •

edited by nhumblot

Loading

nhumblot left a comment

nhumblot May 25, 2021

nhumblot May 25, 2021

fcamblor commented May 26, 2021 •

edited

Loading

francoisBouchet commented May 26, 2021

fcamblor commented May 26, 2021 •

edited

Loading

fcamblor commented Jun 3, 2021

Luwangel commented Jun 7, 2021

fcamblor commented Jun 7, 2021

bilelz Jun 7, 2021

mccob commented Jul 12, 2021

fcamblor commented Jul 13, 2021 •

edited

Loading

Floby left a comment

fcamblor commented Jul 21, 2021

fcamblor commented Jul 24, 2021 •

edited

Loading

Nouveau mode de fonctionnement en prévision de l'éligibilité 18+ : introduction du calendrier des créneaux #233

Nouveau mode de fonctionnement en prévision de l'éligibilité 18+ : introduction du calendrier des créneaux #233

Conversation

fcamblor commented May 25, 2021 • edited by nhumblot Loading

Description

nhumblot left a comment

Choose a reason for hiding this comment

nhumblot May 25, 2021

Choose a reason for hiding this comment

nhumblot May 25, 2021

Choose a reason for hiding this comment

fcamblor commented May 26, 2021 • edited Loading

francoisBouchet commented May 26, 2021

fcamblor commented May 26, 2021 • edited Loading

fcamblor commented Jun 3, 2021

Luwangel commented Jun 7, 2021

fcamblor commented Jun 7, 2021

bilelz Jun 7, 2021

Choose a reason for hiding this comment

mccob commented Jul 12, 2021

fcamblor commented Jul 13, 2021 • edited Loading

Floby left a comment

Choose a reason for hiding this comment

fcamblor commented Jul 21, 2021

fcamblor commented Jul 24, 2021 • edited Loading

fcamblor commented May 25, 2021 •

edited by nhumblot

Loading

fcamblor commented May 26, 2021 •

edited

Loading

fcamblor commented May 26, 2021 •

edited

Loading

fcamblor commented Jul 13, 2021 •

edited

Loading

fcamblor commented Jul 24, 2021 •

edited

Loading