Skip to content

Afficher les notifications à venir dans le tunnel de pose de RDV admin

Vincent Agnano requested to merge feature/admin-rdv-tunnel-notifications into master

Created by: adipasquale

closes #1042 (closed)

nouvelle UX du tunnel de RDV

Screenshot 2021-02-22 at 12 26 17
  • Titre dans la barre noire simplifiés "Créer RDV 2/4" etc
  • étapes clarifiées 1, 2, 3 et 4, de haut en bas, interface type "accordéon" avec l'étape courante dépliée
  • avant on voyait déjà les étapes précédentes, mais pas d'unité par étape il y avait un item par info du RDV, maintenant c'est un item par étape
  • on peut cliquer sur les étapes précédentes, par exemple pour revenir de l'étape 4 à l'étape 1 directement

nouvelle etape 4 Notifications

Screenshot 2021-02-22 at 12 31 02

J'ai essayé de faire relativement générique en réutilisant les helpers de la page admin/users#show

un detail c'est que j'essaie de gerer le cas : 1 RDV avec 2 usagers : le responsable et son proche. Dans ce cas le systeme devrait expliciter que pour l'usager proche on va notifier son responsable, et n'afficher qu'une fois les infos du responsable

Ca implique quelques petits detournements du code dans le rdvwizardform pour introduire la nouvelle etape tout en preservant les warnings sur l'horaire a l'etape 3

Refactos

  • l'héritage comme RdvWizardForm::Step1 < Base devient de la composition avec l'extraction de RdvWizardFormConcern
  • les différentes steps vivent chacun dans un fichier à part maintenant plutôt qu'un fichier commun, pour mieux respecter les conventions Rails
  • suppression de la methode Admin::RdvFormConcern#save qui doit maintenant être définie dans les classes incluant ce concern. C'était en fait déjà le cas, elle était tout le temps surchargée.
  • découpage des vues admin/rdv_wizard_form/step1 etc en plusieurs partials pour génériser une partie du code. Ce n'est pas encore parfaitement satisfaisant je pens equ'on voudrait aller plus loin

Merge request reports