-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Permettre aux usagers de joindre l'organisation avec laquelle ils ont rendez-vous #4899
base: production
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙇 merci beaucoup d’avoir entamé ce chantier et regardé la donnée !
C’est super intéressant de voir que ça pourrait couvrir de nombreux cas.
Je pense qu’il faut sans faute en parler avec Nesserine et avoir son go avant de déployer ça.
Côté UI/UX je n’ai pas un avis très fort mais ça me paraît un peu trop mis en avant dans cette proposition. Est-ce que tu as envisagé d’en faire plutôt un bouton tertiaire ?
je n’affiche pas le numéro ce qui peut être moins accessible en revanche
Ça vaudrait probablement le coup de demander conseil à Téo. J’ai une petite crainte qu’on mette « trop » en avant ces numéros de téléphone mais je pense que ça vaut le coup d’essayer et de voir les retours qu’on a des orgas.
On pourra aussi envisager d’afficher la même chose dans les emails de recap.
ah et j’ai oublié de dire aussi que l’affichage du lien vers le site c’est cool mais peut-être à faire dans un deuxième temps ? je ne sais pas trop quel volume ça couvre et si ça sera vraiment utile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 super, merci beaucoup !
J’ai mis des notes pour d’éventuelles pistes plus tard mais absolument pas bloquantes.
i.fa.fa-exclamation-triangle> | ||
strong Informations supplémentaires : | ||
.pl-3.pt-1 | ||
= instruction_for_rdv_to_html(rdv.motif) | ||
= render "/users/rdvs/file_attente", rdv: rdv | ||
|
||
- if rdv.phone_number || rdv.organisation.website.present? | ||
.fr-mb-2w | ||
h2.fr-h6.fr-mt-3w Contact de #{rdv.organisation.name} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
un détail ici mais on va potentiellement afficher le nom de l’orga et le numéro de téléphone du lieu. On peut avec de l’imagination penser à des cas où ça serait problématique mais c’est vraiment peu probable et on pourra améliorer plus tard.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oui, je pense que c'est raisonnable, dans la mesure où dans la plupart des cas, le lieux et l'organisation sont la même chose (ex : une MDS, une Mairie), et que même si l'organisation a plusieurs lieux qui ont chacun leur numéro de téléphone, le numéro de téléphone du lieux est finalement juste un des différents numéros de téléphone de l'organisation (et c'est celui qui est pertinent pour ce rendez-vous)
app/views/users/rdvs/_rdv.html.slim
Outdated
- if rdv.organisation.website.present? | ||
p | ||
span.fr-icon-global-fill.fr-mr-1w[aria-hidden="true"] | ||
= link_to rdv.organisation.website, rdv.organisation.website, target: "_blank" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on pourrait éventuellement utiliser un texte de lien type « Site web » ou « Site web de #{orga.name} » pour s’éviter des mauvaises surprises de liens super longs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bien vu, je corrige ça
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je n'ai pas de remarque particulière hormis coté UI, mais dans l'idée c'est très cool.
-
Padding : on perd en padding-left, ce qui se comprend si on veut s'aligner avec le CTA, toutefois possible d'avoir le meme écart pour le padding-top ?
-
Ajouter un espacement de 1w entre les pictos et les textes
-
pour le titre "Contact", je me demande si on gagnerait pas en cohérence en ajoutant un titre "détail du rdv" pour les infos au dessus, histoire de bien hierarichiser
-
Dernier point qui pourrait faire l'objet d'une autre PR : ajouter les labels correspondant aux pictos pour une meilleure compréhension de l'info (on a des motifs qui s'appellent "Autres" dans des CDAD, donc sans contexte on peut ne pas comprendre a quoi cela correspond)
Sinon le reste est ok :)
Contexte
Les infos qu'on donne aux usagers sont très minimalistes : on ne donne très peu d'info sur la structure avec laquelle ils ont rendez-vous. Si le lieu a un numéro de téléphone et que le rendez-vous est sur place, on l'indique, mais c'est un cas minoritaire.
On ne donne aucune information sur l'organisation, contrairement à ce qu'on fait lors de la prise de rendez-vous (si un usager cherche un créneau mais n'a aucun résultat, on lui affichera le numéro de téléphone, l'adresse mail et le site web de l'organisation).
Voici un récap de ces différents cas de figure sur metabase : on a plus d'un million de rdvs où le lieu n'a pas de numéro de téléphone mais l'orga en a un.
https://rdv-service-public-metabase.osc-secnum-fr1.scalingo.io/question/222-numeros-de-telephones-des-lieux-et-des-organisations
Il semble très probable que cette impossibilité de contacter l'organisation fait que les usagers finissent par nous contacter à la place (puisque notre page de contact le permet).
Solution
On redirige le flux de demande vers les organisations qui sauront y répondre. D'après ce que je comprends certaines référentes sont réticentes à ce qu'on affiche l'adresse mail de l'organisation, donc cette PR ne propose pas encore ça.