On affiche l’aide au RDV de suivi que si nécessaire #4918
Conversation
AntoineGirard
force-pushed
the
agd/feat-display-follow-up-motifs-sign-in-only-if-necessary
branch
2 times, most recently
from
eae3ec8 to
cebe0e4
Compare
|
🔥🔥🔥 |
AntoineGirard
force-pushed
the
agd/feat-display-follow-up-motifs-sign-in-only-if-necessary
branch
from
cebe0e4 to
9880d17
Compare
There was a problem hiding this comment.
Trop bien ! C'est un très bon pattern de ne pas afficher ce bout d'interface s'il n'est pas nécessaire, et c'est top d'avoir sorti les chiffres pour mieux comprendre le besoin. Je pense qu'on peut restreindre encore plus le scope, mais c'est pas bloquant pour merger.
| @@ -64,6 +64,10 @@ def services | |||
| @services ||= matching_motifs.includes(:service).map(&:service).uniq.sort_by(&:name) | |||
| end | |||
|
|
|||
| def follow_up_motifs? | |||
| @follow_up_motifs ||= Motif.where(service: services).exists?(follow_up: true, deleted_at: nil) | |||
There was a problem hiding this comment.
👍 C'est intéressant d'utiliser Motif.where(service: services) plutôt que matching_motifs. On pourrait même restreindre un peu plus en ajoutant where.not(bookable_by: :agents) et en filtrant sur le territoire (plusieurs territoires peuvent utiliser le même service)
There was a problem hiding this comment.
Yes, j'avais hésité à pousser un peu + loin avec cette histoire de bookable_by et je me disais qu’en première étape, c'était déjà pas mal, mais vu ton retour, je vais peut-être pousser un peu plus :)
spec/requests/prendre_rdv_spec.rb
Outdated
| it "shows a hint to help find a rdv with a referent agent in case the user is looking for follow_up motifs" do | ||
| get root_path(departement: "75", city_code: "75056", latitude: "48.859", longitude: "2.347", address: "Paris 75001") | ||
| expect(response.body).to include("Pour prendre un RDV de suivi avec un de vos agents référent") | ||
| context "when territory has no follow up motifs" do |
There was a problem hiding this comment.
ce contexte est un peu inexact : c'est le service qui n'a pas de motif de follow up. Avec l'implémentation actuelle, si le territoire n'a pas de motifs de follow_up pour ce service mais qu'un autre territoire en a un, on affichera l'invitation à se connecter
There was a problem hiding this comment.
Effectivement, je vais réécrire le contexte pour que ça soit un peu + exacte
AntoineGirard
force-pushed
the
agd/feat-display-follow-up-motifs-sign-in-only-if-necessary
branch
from
ce571ff to
c5c13a7
Compare
AntoineGirard
force-pushed
the
agd/feat-display-follow-up-motifs-sign-in-only-if-necessary
branch
from
c5c13a7 to
3129a96
Compare
AntoineGirard
deleted the
agd/feat-display-follow-up-motifs-sign-in-only-if-necessary
branch
Review app
Contexte
Dans le parcours de prise de RDV, nous avons un bandeau qui incite les usagers qui souhaitent prendre un RDV de suivi avec leur agent référent à se connecter.
Ce bandeau ne concerne que :
L’idée est donc d’afficher le bandeau que si on est dans un contexte dans lequel il peut potentiellement y avoir un RDV de suivi.
Solution
Conséquence :