Skip to content

better preserve agent context in rdvs, plage ouvertures and absences views

Vincent Agnano requested to merge feature/preserve-agent-context-rdv-show into master

Created by: adipasquale

https://trello.com/c/95YCduVA/1164-perte-de-contexte-suite-%C3%A0-suppression-dun-rdv-dans-lagenda

ℹ️ lisible commit par commit ℹ️ @yaf hésite pas à merger en mon absence, c'est assez safe

ce n'est pas parfaitement trivial de préserver le contexte de l'agent sur toutes les pages, il faut bien s'assurer de le garder partout.

Pages RDVs

on peut arriver depuis l'agenda mais aussi depuis la liste des RDVs, ou bien les users show. Il faut donc gérer le cas où on est dans le contexte d'un agent et le contexte sans. J'ai donc passé l'agent en param get agent_id et il faut bien faire attention de le passer partout et le retuiliser quand on redirige, par ex apres un create.

Pages Absences et POs

On peut seulement arriver depuis l'agenda (je crois), donc c'est un peu plus simple. La petite complexité réside dans le fait qu'on utilise des routes scopées différemment pour les différentes actions :

  • http://localhost:5000/admin/organisations/1/agents/2/absences pour la vue index

mais

  • http://localhost:5000/admin/organisations/1/absences pour la création ou la suppression

J'ai commencé à refacto tout scoper dans agent mais j'ai eu un doute que ce soit la bonne direction donc j'ai arrêté et pris la route du moindre effort.

il faut donc gérer la récupération de l'agent depuis le param ou depuis la plage d'ouverture.

A côtés

J'en ai profité pour corriger quelques trucs :

  • rajouté le breadcrumb sur la page rdvs#edit
  • amélioré le breadcrumb de la page rdvs#show (selon de quel agent on vient)
  • retiré l'emoji espion de l'historique des actions (il ne plait pas ni ne marche tres bien selon les fonts dispos)

Merge request reports