Skip to content
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

Première version de la prise de rdv par intégration #5006

Merged
merged 23 commits into from
Jan 29, 2025

Conversation

victormours
Copy link
Contributor

@victormours victormours commented Jan 23, 2025

Contexte

On passe une première version imparfaite de la prise de rdv par api pour ne pas bloquer les équipes de Démarches Simplifiées, Mon Suivi Social et de la Coop de la Médiation Numérique pendant que je pars en congés.

Solution

C'est une version basique
On a hésité entre deux interfaces possibles après le clic dans le calendrier :

  • un choix de motif comme dans la branche rdv-plan-calendar-first-alt
  • un choix de modalité puis de motif comme proposé dans cette branche.

Trois arguments ont motivés ce choix :

  • le choix de modalité est compréhensible par quelqu'un qui n'est pas habitué à notre modèle de données, contrairement à un choix de motif qui force un certain location type
  • le choix de motif puis de lieu si nécessaire rendait difficile le choix de durée du rdv
  • les rich radio buttons permettent de mieux visualiser les différentes modalités que le select de motifs

On remarquera que le premier argument est le plus fort, puisque les deux autres pourraient sans doute être levés par un peu de travail supplémentaire sur l'UI.

De toutes façons, notre objectif reste de faire des tests utilisateurs la semaine du 10 février pour avoir les retours des agents.

@victormours victormours marked this pull request as draft January 23, 2025 07:58
app/javascript/stylesheets/rdv_plan.scss Outdated Show resolved Hide resolved
app/policies/agent/rdv_plan_policy.rb Outdated Show resolved Hide resolved
app/views/agents/rdv_plans/_modalites_field.html.slim Outdated Show resolved Hide resolved
lieu: lieu,
organisation: organisation
)
expect(user.reload.email).to eq "[email protected]"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

faire un expect sur les deux mails

Copy link
Contributor

@francois-ferrandis francois-ferrandis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vu ensemble, en confiance pour merger ça. 👍

@victormours victormours marked this pull request as ready for review January 24, 2025 14:56
@francois-ferrandis francois-ferrandis merged commit b978726 into production Jan 29, 2025
15 checks passed
@francois-ferrandis francois-ferrandis deleted the rdv-plan-wizard branch January 29, 2025 21:47
Copy link

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ ActiveRecord::RecordInvalid: La validation a échoué : Email est déjà utilisé (ActiveRecord::RecordInvalid) Agents::RdvPlansController#create_rdv View Issue

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants