Skip to content

add users.notify_by_sms and email settings

Vincent Agnano requested to merge feature/user-notification-settings into master

Created by: adipasquale

https://trello.com/c/onN13lua/1175-d%C3%A9sactiver-les-notifications-pour-un-usager

Screenshots

users#edit

user-edit

users rdv tunnel

users-rdv-tunnel

admin/users#show

admin-user-show

admin/users#edit

admin-user-edit-enabled

L'admin ne peut pas modifier les prefs de notifs une fois le compte confirmé :

admin-user-edit-disabled

admin/rdvs#show

admin-rdv-show

Implementation tweaks

J'ai du rajouter encore un peu de complexité dans le JS du formulaire des users dans l'admin. ce JS est responsable de basculer du mode "usager responsable" vers le mode "usager proche". Il gère la visibilité des bons champs selon le mod, et maintient à jour les attributs disabled sur les différents inputs pour s'assurer que le formulaire soumettra les bons champs.

L'utilisation de champs de types checkbox pour les nouvelles preferences de notifs fait que rails rajoute un hidden input avec la valeur 0 pour quand la checkbox n'est pas coché. Ce input n'a pas les memes classes et attributs que le principal, donc pas data-togglable et donc il n'était pas bien géré par le formulaire actuel.

Une alternative pour ne pas avoir à gérer ce JS supplémentaire c'est de ne pas utiliser des checkboxes mais par exemple des radio buttons OUI / NON pour les deux options. J'avais commencé par ça mais c'est vraiment pas terrible en UX je trouve.

TODO

  • test on review app

Plus tard

Ca serait bien de rajouter du contenu sur admin/rdv_wizard#step3 pour expliquer quelles notifs s'appretent a partir a la creation du RDV (aucune / sms / email / les deux). ce n'est pas mega evident ou faire apparaitre cette info donc je prefere le faire plus tard dans une PR a part.

later

Merge request reports