profile
viewpoint
Frédéric Camblor fcamblor 4sh Villenave d'Ornon, France http://fcamblor.wordpress.com

CovidTrackerFr/vitemadose-front 75

Interface utilisateur de l'outil ViteMaDose

fcamblor/backbone-fundamentals 1

A creative-commons book on Backbone.js for beginners and advanced users alike

CovidTrackerFr/vitemadose-firebase 0

Gestion des notifications pour les applications iOS et Android de ViteMaDose

fcamblor/AlertifyJS 0

A javascript framework for developing pretty browser dialogs and notifications.

fcamblor/angular-typescript-webpack 0

My angular typescript webpack architecture

fcamblor/angular2-codelab 0

Angular 2 Codelab: Let's develop an Angular 2 application...

fcamblor/awesome-java 0

A curated list of awesome Java frameworks, libraries and software. Inspired by awesome-python.

PR opened nicmarti/cfp-devoxx

No comitee email on pub comment

This PR fixes a regression introduced during #259 : no emails are sent to speakers when a public comment is posted by comitee.

A special admin-only endpoint has been set-up to recover (re-send) these emails : 1/ Navigate to https://cfp.devoxx.fr/admin/fixUnsentSpeakerEmailsAfterPublicComment?start=2021-11-01T00:00:00Z&end=2021-12-01T12:00:00Z&sendEmail=false and check that every emails are looking good 2/ If everything is good, simply change sendEmail=false to sendEmail=true to send the email to speakers.

+55 -14

0 comment

3 changed files

pr created time in a day

push eventfcamblor/cfp-devoxx

fcamblor

commit sha 11e12b5a0d0cf7bfe4689ee98018648ea1a3fc6a

provided (admin-only) backoffice endpoint in order to recover non-sent emails to speakers

view details

push time in a day

create barnchfcamblor/cfp-devoxx

branch : no-comitee-email-on-pub-comment

created branch time in a day

push eventfcamblor/cfp-devoxx

Frédéric Camblor

commit sha dd867d5669d12ef456b4492465c7395c08386b6c

More digest fixes (#261) * using different labels for AutoWatch.AFTER_INTERACTION between cfp & gt users (we shouldn't mention comments on GTs) * introduced AutoWatch.AFTER_COMMENT cfp-specific option, allowing to autowatch on comments (only) * removing public comment event from allowed GT events in notification preferences, as GT don't have access to public comments. Checked that provided eventIds by GTs are valid (filtering them out otherwise) * added some space after icons on buttons * using conference descriptor's timezone when showing comment's publication date * added missing flash-based messages on preferences page, in order to show save success and/or potential errors * fixed regression where we were not sending emails to speaker when a new public comment was posted

view details

push time in a day

pull request commentnicmarti/cfp-devoxx

More digest fixes

Just added a fix for an important regression : speaker are currently not receiving public comment messages

I will do a follow up fix tonight in order to send emails to speakers for proposals having public comments.

fcamblor

comment created time in a day

push eventfcamblor/cfp-devoxx

fcamblor

commit sha b4c70e43eb50ad6640d13e61381fdd40842e1718

fixed regression where we were not sending emails to speaker when a new public comment was posted

view details

push time in a day

push eventfcamblor/cfp-devoxx

fcamblor

commit sha fdf039a07bb3e86be58ddbc8538552101f87c55e

introduced digest event for resetted votes

view details

fcamblor

commit sha e531a528b55164613b474fff70e9e1487bb90642

displaying previous vote when vote reset has been detected

view details

push time in a day

PR opened nicmarti/cfp-devoxx

Helping review after resetted votes

This PR aims at helping reviewer retrieve their previous vote when a Vote resetted event happens.

These changes are two fold :

  • Digest section to votes resetted is added (with dedicated event to track it) CFP_Admin_-<em>Email_Digests_Queue</em>-<em>Devoxx_France_2022 CFP_Admin</em>-_Devoxx_France_2022

  • Previous vote is shown under votes : <em>CFP__GMZ-4258_qweqeqweq</em>-_Devoxx_France_2022

This is only shown if a vote was casted prior to any reset, and no new vote has been casted since latest reset

+56 -11

0 comment

9 changed files

pr created time in a day

push eventfcamblor/cfp-devoxx

fcamblor

commit sha 891b90573e7a247a52ae79e4c43b89a6d4dd9daa

to cherry-pick

view details

fcamblor

commit sha e334f581618a5071408c74c92133a39ac9fadd9c

displaying previous vote when vote reset has been detected

view details

push time in a day

create barnchfcamblor/cfp-devoxx

branch : helping-review-after-resetted-votes

created branch time in a day

push eventCovidTrackerFr/vitemadose-front

fcamblor

commit sha d6aa6e690df78fdd6396bb4ee5c81c0775b1ecfa

centralized default search type into a constant

view details

fcamblor

commit sha b03a8d880856cdd3cbdd7a67adba63b4f7751c2a

fix space in communes-import call in readme

view details

fcamblor

commit sha 0b46dd20b10dba5c58f64edd0803f47aa129f7a6

commented out 18-55 search

view details

fcamblor

commit sha b754fdf38c2f992e281259e5096c5278c5ccc087

implemented 2 new search types : 1+2 shots, and booster shot

view details

fcamblor

commit sha 5e5f27601db0cdbfc246ad4ffb72eb8e2dde72df

re-generated sitemaps to take into consideration new search types

view details

Frédéric Camblor

commit sha cfb105510a175861ebca1ee360d2a118a6b3b57f

Merge pull request #273 from CovidTrackerFr/dose-rappel-et-1ere-dose Dose rappel et 1ere dose

view details

fcamblor

commit sha 1f21ba4fbdb708162ffbc0ada80e1acae49ea1d3

Merge branch 'dev'

view details

push time in 2 days

push eventCovidTrackerFr/vitemadose-front

fcamblor

commit sha d6aa6e690df78fdd6396bb4ee5c81c0775b1ecfa

centralized default search type into a constant

view details

fcamblor

commit sha b03a8d880856cdd3cbdd7a67adba63b4f7751c2a

fix space in communes-import call in readme

view details

fcamblor

commit sha 0b46dd20b10dba5c58f64edd0803f47aa129f7a6

commented out 18-55 search

view details

fcamblor

commit sha b754fdf38c2f992e281259e5096c5278c5ccc087

implemented 2 new search types : 1+2 shots, and booster shot

view details

fcamblor

commit sha 5e5f27601db0cdbfc246ad4ffb72eb8e2dde72df

re-generated sitemaps to take into consideration new search types

view details

Frédéric Camblor

commit sha cfb105510a175861ebca1ee360d2a118a6b3b57f

Merge pull request #273 from CovidTrackerFr/dose-rappel-et-1ere-dose Dose rappel et 1ere dose

view details

push time in 2 days

PR merged CovidTrackerFr/vitemadose-front

Reviewers
Dose rappel et 1ere dose

Cette Pull Request est

  • Une nouvelle fonctionnalité (dans le prolongement de la réflexion sur #270)

Checklist

  • Cette PR vise la branche dev
  • Elle n'est pas en conflit avec la branche dev

Description

Rajout d'un filtre permettant de rechercher soit les doses de rappel, soit les 1ère doses (et aussi, mais on ne l'évoque pas pour éviter du bruit/de l'ambiguité, les 2ndes doses)

Visible ici : https://dev.vitemado.se/dose-rappel-et-1ere-dose/ Vaccination_COVID-19_à_Villenave-d_Ornon_33140

Quelques précisions :

  • Par rapport au choix de la dose de rappel par défaut :

88% de la population vaccinnable a actuellement un chemin vaccinal complet (et 2% a un chemin vaccinal entamé) (source: vaccin tracker) => il reste seulement 10% de la population vaccinable non vaccinée, et ce n'est pas à priori celle qui va être le plus susceptible d'aller sur VMD pour rechercher une dose de vaccin.

C'est pour ces raisons que j'encouragerais à pré-selectionner le choix "dose de rappel" lors de la recherche.

  • Par rapport au fait de ne pas afficher que le choix "1ère dose de vaccin" couvre aussi les secondes doses :

=> La notion de "2ème dose" disparaît car elle est très ambigüe vis à vis de ceux qui ont eu le COVID (et donc 1 seule dose) ou ceux qui ont reçu un JJ à 1 seule dose aussi. De plus, les rdv 2nde dose étant très généralement pris au moment où on se fait injecter la 1ère dose, je pense que vraiment très peu de personnes viendront sur VMD à la recherche d'une 2nde dose (inutile, donc, de venir rajouter du bruit/ambiguité avec cette notion dans le filtre de recherche).

On peut toutefois garder le filtre sur le tag 1+2 lorsqu'on sélectionne 1ère dose de vaccin car je ne pense pas qu'il y ait beaucoup de créneaux qui soient compatible 2ème dose sans être compatibles 1ère dose.

  • Par rapport au fait qu'on n'affichait pas les créneaux indéterminés :

Je pense qu'on va pouvoir laisser tomber les indéterminés car nous n'en avons que très peu actuellement en prod (les 2 plateformes concernées sont des plateformes qui sont en cours de migration (une fois qu'elles auront été migrées, les doses de rappel apparaîtront)) : Vaccination_COVID-19_en_Aisne_02

PS:

  • Merci @nhumblot pour le taf fait en amont :)
  • Merci @grubounet pour le taf fait sur le back ❤️
+107518 -35830

0 comment

107 changed files

fcamblor

pr closed time in 2 days

pull request commentCovidTrackerFr/vitemadose-front

Première implémentation sélecteur dose rappel

@rozierguillaume oui on pourrait (pour moi, on est bons sur #273)

Coté app mobile, on est pas loins d'être bons aussi je pense (cc @michgauz)

@nhumblot @grubounet @Luwangel je vous ai mis reviewers sur #273, vous pouvez aller checker et me dire si c'est OK pour vous ?

nhumblot

comment created time in 2 days

issue commentCovidTrackerFr/vitemadose-front

Calendrier : ligne blanche verticale sur iOS

Dans le même ordre d'idée : Vaccination_COVID-19_à_Villenave-d_Ornon_33140

rozierguillaume

comment created time in 2 days

pull request commentCovidTrackerFr/vitemadose

[Doctolib] - corrections 3e doses

Pour info, ça semble mieux (encore beaucoup de 1ere/2eme dose en proportion par rapport aux doses de rappel je trouve) Vaccination_COVID-19_en_Aisne_02

grubounet

comment created time in 2 days

PR opened CovidTrackerFr/vitemadose-front

Dose rappel et 1ere dose

Cette Pull Request est

  • Une nouvelle fonctionnalité (dans le prolongement de la réflexion sur #270)

Checklist

  • Cette PR vise la branche dev
  • Elle n'est pas en conflit avec la branche dev

Description

Rajout d'un filtre permettant de rechercher soit les doses de rappel, soit les 1ère doses (et aussi, mais on ne l'évoque pas pour éviter du bruit/de l'ambiguité, les 2ndes doses)

Visible ici : https://dev.vitemado.se/dose-rappel-et-1ere-dose/ Vaccination_COVID-19_à_Villenave-d_Ornon_33140

Quelques précisions :

  • Par rapport au choix de la dose de rappel par défaut :

88% de la population vaccinnable a actuellement un chemin vaccinal complet (et 2% a un chemin vaccinal entamé) (source: vaccin tracker) => il reste seulement 10% de la population vaccinable non vaccinée, et ce n'est pas à priori celle qui va être le plus susceptible d'aller sur VMD pour rechercher une dose de vaccin.

C'est pour ces raisons que j'encouragerais à pré-selectionner le choix "dose de rappel" lors de la recherche.

  • Par rapport au fait de ne pas afficher que le choix "1ère dose de vaccin" couvre aussi les secondes doses :

=> La notion de "2ème dose" disparaît car elle est très ambigüe vis à vis de ceux qui ont eu le COVID (et donc 1 seule dose) ou ceux qui ont reçu un JJ à 1 seule dose aussi. De plus, les rdv 2nde dose étant très généralement pris au moment où on se fait injecter la 1ère dose, je pense que vraiment très peu de personnes viendront sur VMD à la recherche d'une 2nde dose (inutile, donc, de venir rajouter du bruit/ambiguité avec cette notion dans le filtre de recherche).

On peut toutefois garder le filtre sur le tag 1+2 lorsqu'on sélectionne 1ère dose de vaccin car je ne pense pas qu'il y ait beaucoup de créneaux qui soient compatible 2ème dose sans être compatibles 1ère dose.

  • Par rapport au fait qu'on n'affichait pas les créneaux indéterminés :

Je pense qu'on va pouvoir laisser tomber les indéterminés car nous n'en avons que très peu actuellement en prod (les 2 plateformes concernées sont des plateformes qui sont en cours de migration (une fois qu'elles auront été migrées, les doses de rappel apparaîtront)) : Vaccination_COVID-19_en_Aisne_02

PS:

  • Merci @nhumblot pour le taf fait en amont :)
  • Merci @grubounet pour le taf fait sur le back ❤️
+107518 -35830

0 comment

107 changed files

pr created time in 2 days

pull request commentCovidTrackerFr/vitemadose-front

Première implémentation sélecteur dose rappel

J'ai poussé la version rappel / 1+2 ici : https://dev.vitemado.se/dose-rappel-et-1ere-dose/

J'aimerais bien rajouter des tooltips pour préciser ce que cela concerne, mais je n'ai plus le temps là (à voir si on rajoute les tooltips dans un second temps ou si on attend de les avoir pour MEP)

Par tooltip, on a 2 options :

  • Soit faire un vrai tooltip (comme on avait fait sur l'onglet chronodose, basé sur la lib tippyjs) au clic sur l'option
  • Soit faire des options un peu "riches" avec un titre en gras et un texte explicatif en plus petit, comme l'avait proposé @rozierguillaume (moi j'aime bien, mais j'ai un peu peur que ça grignote pas mal de place sur mobile) image (C'est pour l'exemple, l'option "tous" n'y sera pas et je pense qu'il faudra un peu retravailler les textes pour préciser certains points, notamment concernant les personnes ayant eu le covid)
nhumblot

comment created time in 2 days

create barnchCovidTrackerFr/vitemadose-front

branch : dose-rappel-et-1ere-dose

created branch time in 2 days

pull request commentCovidTrackerFr/vitemadose-front

Première implémentation sélecteur dose rappel

@rozierguillaume 88% de la population vaccinnable a actuellement un chemin vaccinal complet (et 2% a un chemin vaccinal entamé) => il reste seulement 10% de la population vaccinable non vaccinée, et ce n'est pas à priori celle qui va être le plus susceptible d'aller sur VMD pour rechercher une dose de vaccin.

C'est pour ces raisons que j'encouragerais à pré-selectionner le choix "dose de rappel" lors de la recherche.

nhumblot

comment created time in 3 days

pull request commentCovidTrackerFr/vitemadose-front

Première implémentation sélecteur dose rappel

Je ne vois pas trop la plu-value du "Tous" très honnêtement (les filtres rappel et 1+2 étant disjoints au niveau parcours vaccinal)

Après quelques discussions avec Guillaume en début d'aprem, j'ai (encore) un peu évolué dans ma réflexion pour finir par me diriger vers un :

Vous recherchez une : Dose de rappel | 1ère dose de vaccin

=> La notion de "2ème dose" disparaît car elle est très ambigüe vis à vis de ceux qui ont eu le COVID (et donc 1 seule dose) ou ceux qui ont reçu un JJ à 1 seule dose aussi. De plus, les rdv 2nde dose étant très généralement pris au moment où on se fait injecter la 1ère dose, je pense que vraiment très peu de personnes viendront sur VMD à la recherche d'une 2nde dose (inutile, donc, de venir rajouter du bruit avec cette notion dans le filtre de recherche).

On peut toutefois garder le filtre sur le tag 1+2 lorsqu'on sélectionne 1ère dose de vaccin car je ne pense pas qu'il y ait beaucoup de créneaux qui soient compatible 2ème dose sans être compatibles 1ère dose.

nhumblot

comment created time in 3 days

pull request commentCovidTrackerFr/vitemadose-front

Première implémentation sélecteur dose rappel

Je pense qu'on va pouvoir laisser tomber les indéterminés car nous n'en avons que très peu actuellement en prod : image

(oui, il y a des trucs étranges dans ce tableau, notamment au niveau des doses de rappel Doctolib... @grubounet jètera un oeil ce soir)

nhumblot

comment created time in 3 days

Pull request review commentCovidTrackerFr/vitemadose

[3e doses] - Adaptation tags à la demande du front

 def fetch_slots(request: ScraperRequest, creneau_q=DummyQueue, client: httpx.Cli     return maiia.fetch(request)  +def get_vaccine_type_from_name(motive_name):+    if not motive_name:+        return None+    dose = None+    if "première" in motive_name.lower() or "1" in motive_name.lower():

@grubounet Je pense qu'il faudrait inclure la valeur premiere (sans accent), et, juste en-dessous, troisieme, dans les cas à tester (j'imagine que les gens peuvent omettre l'accent ... ça serait ballot de pas capter ces doses dans ces cas de figure :-) )

grubounet

comment created time in 3 days

PullRequestReviewEvent

pull request commentCovidTrackerFr/vitemadose-front

Première implémentation sélecteur dose rappel

Pour info, suite à la mise à dispo des nouveaux tags par le back (cf https://github.com/CovidTrackerFr/vitemadose/pull/661) je viens de déployer une version en parallèle de celle de @nhumblot et présentant l'option 1️⃣ évoquée ci-dessus : https://dev.vitemado.se/dose-rappel-et-indeterminee/

On a donc : 1️⃣ => https://dev.vitemado.se/dose-rappel-et-indeterminee/ 2️⃣ => https://dev.vitemado.se/dose-rappel/

nhumblot

comment created time in 3 days

pull request commentCovidTrackerFr/vitemadose

[3e doses] - Adaptation tags à la demande du front

Trop merci :-)

grubounet

comment created time in 3 days

push eventCovidTrackerFr/vitemadose-front

fcamblor

commit sha 11640b16da7ecb0feb88cad231332b71dba6b73d

getting rid of the 18_55 search type (commented it out)

view details

fcamblor

commit sha 3b80cca03be4cbeb0758a836354437f81f62004b

re-generated sitemaps with new dose_1_ou_2 / dose_indeterminee and without 18_55 search types

view details

push time in 3 days

create barnchCovidTrackerFr/vitemadose-front

branch : dose-rappel-et-indeterminee

created branch time in 3 days

pull request commentCovidTrackerFr/vitemadose-front

Première implémentation sélecteur dose rappel

Entretemps j'ai un peu changé mon fusil d'épaule, pour me diriger finalement vers un triple choix doses de rappel / doses 1+2 / undéterminé car le "Tous" ne me paraît pas vraiment utile (on se met à "noyer" les doses #1/#2/indéterminées dans la "masse" de "tous les créneaux dispos")

Ça donnerait un truc comme ça : Image_28_11_2021_23_40_collée

Si on se cantonne à un simple double choix dose de rappel / tous je pense qu'il faut être très explicite sur ce qu'il y a derrière le "tous" afin de bien faire comprendre que dans le lot, il y a des créneaux où on n'a pas pu déterminer le chemin vaccinal (et, du coup, où ça pourrait être un créneau compatible avec les doses de rappel). Ça donnerait un truc comme ça : Vaccination_COVID-19_à_Villenave-d_Ornon_33140

Ça m'intéresse d'avoir vos feedback sur les 2 options :

  • 1️⃣ pour l'option avec les 3 choix doses de rappel / doses 1+2 / undéterminé
  • 2️⃣ pour l'option avec les 2 choix doses de rappel / tous
nhumblot

comment created time in 3 days

pull request commentCovidTrackerFr/vitemadose-front

Première implémentation sélecteur dose rappel

J'ai checké le code et ça me paraît bien.

Il faut simplement voir si on conserve l'implem actuelle qui présente des failles (on ne prend pas en compte les 2ndes doses, et on ne laisse plus la possibilité de voir "tous" les créneaux) ou si on ne se dirigerait pas plutôt vers une dichotome doses identifiées (avec certitude) comme doses de rappel / tous les créneaux, y compris ceux dontle numéro de dose est inconnu

nhumblot

comment created time in 4 days

more