add users.notify_by_sms and email settings
Created by: adipasquale
https://trello.com/c/onN13lua/1175-d%C3%A9sactiver-les-notifications-pour-un-usager
Screenshots
users#edit
users rdv tunnel
admin/users#show
admin/users#edit
L'admin ne peut pas modifier les prefs de notifs une fois le compte confirmé :
admin/rdvs#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.