Skip to content

Améliorations du bloc de RDVs précédents sur la vue RDV admin

Vincent Agnano requested to merge feature/improve-rdvs-previous-rdvs into master

Created by: adipasquale

closes #1031 (closed)

Changement visuel

Le bloc intitulé "rendez-vous précédents" apparaît maintenant dans la carte de l'usager plutôt que dans une carte indépendante, pour exprimer le fait que ce sont les RDVs de cet usager dont on parle (utile si plusieurs usagers).

stitched_2021_03_01-10_19_38

Changement de scope temporel

Jusqu'ici le filtre "rdvs précédents" se faisait sur base de l'heure actuelle, on parlait des RDVs dans le passé de l'usager. Or, si je suis par exemple en train de regarder la page d'un RDV qui a eu lieu il y a un mois, et que je vois un rdv qui a eu lieu il y a 1j dans un bloc "rdvs précédents" ça n'a pas trop de sens à mon avis. J'ai changé ce filtre temporel, pour qu'on n'affiche ici que les RDVs précédents par rapport au RDV regardé.

Les "RDVs précédents" au sens ceux dans le passé sont toujours visible sur les fiches usagers, mais plus sur les pages RDVs.

Déplacement du code

J'ai déplacé la méthode User#previous_rdvs_ordered_and_truncated dans un nouveau presenter Admin::RdvUserPresenter#previous_rdvs_truncated (au sens un RDV dans le contexte d'un user de ce RDV).

La méthode de User était utilisée uniquement une fois, pour ce cas d'usage. Elle me paraît très spécifique et concerne en fait plutôt des RDVs qu'un user. Je pense aussi que tout ce qu'on peut sortir du modèle User mérite de l'être car ce modèle est déjà trop long selon le linter.

Ajout d'un lien s'il y a plus de 5 RDVs précédents

Screenshot 2021-03-01 at 10 43 44

Merge request reports