betagouv / recosante

6 stars 8 forks source link

Mettre en place un process pour les personnes n'ayant pas activés les notifications #123

Open florianpanchout opened 2 years ago

florianpanchout commented 2 years ago

Apparemment les gens abonnés aux notifications Marie ne l'a pas reçu ce matin.

l-vincent-l commented 2 years ago

Marie n’a pas de données d’abonnements webpush associée à son compte.

J’avais remarqué lors du dev que de temps en temps les infos d’abonnement n’étaient pas envoyées. Je n’ai jamais réussi à reproduire le cas de manière stable, je n’ai pas réussi à faire de capture pour te montrer, ni trouver le bon cas qui ne fonctionne pas. Je pensais que c’était du au fait du développement en local.

Ceci dit, ça c’était pour le cas de Marie, mais je n’ai pas non plus reçu de notifications ce matin. Je continue de creuser pour les autres personnes abonnées.

l-vincent-l commented 2 years ago

Je l’ai reçu pour ma part de matin. Je ne pense pas qu’il y ait d’erreur d’envoi de notification

mariegasnier1 commented 2 years ago

D'accord, étrange. Comment savoir si d'autres utilisateurs sont dans mon cas ?

florianpanchout commented 2 years ago

Prêt de la moitié des utilisateurs inscrits aux notifications sont dans ce cas (68 sur 151). Soit parce qu'ils n'ont pas accepté les notifications, soit parce qu'ils se sont inscrit via iframe (et donc on ne peut même pas leur demander d'accepter).

La solution serait de leur envoyer un mail avec un lien vers une page leur permettant d'activer les notifications sur leur device. Cette page existe déjà, il faut faire le mail et programmer l'envoi

mariegasnier1 commented 2 years ago

D'accord. Et pour les prochains utilisateurs à s'inscrire aux notifications, on pourra faire en sorte de demander l'activation lors du process d'inscription ?

florianpanchout commented 2 years ago

Pas pour ceux en iframe. Et pour les autres qui ne l'acceptent pas on ne peut pas les forcer.

MarinePurson commented 2 years ago

Concernant le mail d'info, c'est prévu dans les mails de relance, on s'était dit qu'on s'en chargeait cette semaine.

florianpanchout commented 2 years ago

J'ai ajouté un lien pour activer les notifications dans les mails de confirmation d'inscription, afin que l'utilisateur puisse s'abonner sur l'appareil de son choix : https://my.sendinblue.com/camp/template/1452/message-setup?editor=v6 https://my.sendinblue.com/camp/template/1453/message-setup?editor=v6

Et j'ai créé un template pour relancer les utilisateurs qui n'ont pas activé les notifications : https://app.sendinblue.com/editor/newsletters/1537

(Il est probablement possible de faire un wording un peu plus clair)

MarinePurson commented 2 years ago

J'ai relu le mail sur l'activitation des notifs, c'est bon pour moi

annepoirot commented 2 years ago

Hello ! De mon côté ça ne fonctionne pas, sur mobile j'ai un message d'erreur lorsque je clique sur "activer les notifications sur cet appareil". Sur l'ordinateur je n'ai pas ce message d'erreur mais je ne le reçois pas pour autant... On peut peut-être créer une autre issue ou voir ça là, tout me va.

florianpanchout commented 2 years ago

En attente de correction de https://github.com/betagouv/recosante/issues/136 avant d'envoyer le mail de relance

annepoirot commented 2 years ago

Du coup on relancerai tout le monde pour préciser aussi la différence android/iOS

florianpanchout commented 2 years ago

Ça rentre dans le cadre du mail de relance non ? Il faudra modifier juste en modifier le contenu (vu ici : https://github.com/betagouv/recosante/issues/137)