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

Invitations RDVI vers des motifs invisibles #4739

Closed
adipasquale opened this issue Oct 21, 2024 · 1 comment
Closed

Invitations RDVI vers des motifs invisibles #4739

adipasquale opened this issue Oct 21, 2024 · 1 comment

Comments

@adipasquale
Copy link
Contributor

ℹ️ Contexte

Je m’intéresse à l’erreur https://sentry.incubateur.net/organizations/betagouv/issues/126102 , des ActiveRecord::RecordNotFound sur Users::RdvsController#show.

Une partie importante de ces erreurs arrive avec un referer de /users/user_name_initials_verification/new, cf cette requête Sentry Discover.

Il me semble que cette route n’intervient que pour les invitations émises par RDVI.

En regardant certains cas j’ai vu plusieurs fois le cas d’un RDV pour un motif « invisible » càd pour lequel l’usager n’est pas censé pouvoir accéder ni modifier ni voir ni rien du tout.
Si j’ai bien compris, il faudrait empêcher d’envoyer des invitations vers ces rdvs de motifs invisibles, ou alors ne pas afficher de lien dans les SMS / mails pour accéder à ces RDV.

J’en profite pour corriger le comportement de la policy qui est incohérent entre sa scope et les méthodes individuelles dans #4738 - mais ça ne va pas résoudre l’erreur du tout.

@adipasquale
Copy link
Contributor Author

fermé en faveur de l’issue pré-existante #3473

@github-project-automation github-project-automation bot moved this from 🔖 Ready to ✅ Done in RDV Service Public Oct 22, 2024
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

No branches or pull requests

1 participant