Skip to content

move Agent select to left menu planning

Vincent Agnano requested to merge feature/left-menu-agent-select into master

Created by: adipasquale

https://trello.com/c/zE9cMLs2/1088-d%C3%A9placer-le-contexte-de-lagent-%C3%A0-visualiser-dans-le-menu-lat%C3%A9ral-le-persister

Screenshot_2020-11-11_at_10 59 02

Comportement

Lorsqu'on est sur une route hors-planning, par ex un users#show:

  • l'agent selectionné dans le select est toujours l'agent courant
  • si on clique sur un des liens Agenda, Plages d'ouvertures, Absences, on arrive sur la vue de l'agent courant
  • si on sélectionne un autre Agent dans le select, on arrive sur l'Agenda de cet agent

Lorsqu'on est sur une route dans le planning :

  • L'agent sélectionné à gauche reste persisté entre toutes les routes au sein de ce sous-groupe du planning
  • Si je suis sur une vue index (par ex absences#index) ou new (par ex plages_ouvertures#new) et que je change d'agent a gauche, je reviens bien vers la meme vue courante (index ou new) avec le nouvel agent selectionné. Ce n'est pas vrai sur les routes edit car l'id de la resource ne serait plus valide
  • Si je suis sur une vue edit ou autre et que je change d'agent, j'arrive sur l'agenda de cet agent.

Détails :

  • je détourne un peu l'usage de content_for pour passer le nom du path helper a utiliser dans le select de changement d'agent. content_for est plutot prevu pour passer du contenu, pas des symboles ou des fonctions. Je n'ai pas trouvé de solution plus élégante. J'ai envisagé un moment de changer le agent_id du path courant en JS mais c'est encore pire je pense, et plus risqué - la au moins on sait ce qui va se passer dans tous les cas.
  • j'en profite pour supprimer une route inutile sur l'agenda qui passait l'agent id en query param plutot que dans le path
  • j'ai retouché le CSS du menu latéral pour bien aligner les chevrons à droite

TODO

  • il faudrait s'occuper du sujet du lien Liste des RDVs et le breadcrumb associé avant, la je l'ai purement supprimé mais ca casse le breadcrumb
  • pour simplifier la transition pour les agents habitués a l'interface actuelle, est-ce que pendant une semaine ou deux on affiche un message dans le header bar ou se trouvait jusqu'ici le select avec un message "agent sélectionnable à gauche"
  • valider la vue responsive => fait dans l'autre PR qui revoit tout le menu de gauche, il faudra merger

Merge request reports