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

Ne pas proposer le service "Secrétariat" dans le formulaire de motif #4207

Merged

Conversation

francois-ferrandis
Copy link
Contributor

Un motif n'est jamais lié à ce service, cf. Motif#not_associated_with_secretariat

J'ai appris ça en lisant cette fiche notion sur le formulaire de motif :
https://www.notion.so/rdvs/Retravailler-le-formulaire-de-cr-ation-de-motif-22d65b065ed44c8b9cddb9ba86774a15

Et par pure coïncidence je viens de tomber sur not_associated_with_secretariat, donc je sors ce quick win (je l'espère, je ne vois pas de piège 🤞 ).

Avant

image

Après

image

Checklist

Avant la revue :

  • Préparer des captures de l’interface avant et après
  • Nettoyer les commits pour faciliter la relecture
  • Supprimer les éventuels logs de test et le code mort

Revue :

  • Relecture du code
  • Test sur la review app / en local

Un motif n'est jamais lié à ce service, cf. `Motif#not_associated_with_secretariat`
@francois-ferrandis francois-ferrandis self-assigned this Mar 27, 2024
@NesserineZarouri
Copy link
Contributor

Bien vu 👍. Sachant qu'on conserve la possibilité d'ouvrir un RDV pour le secrétariat avec la case prévue à cet effet

Copy link
Contributor

@victormours victormours left a comment

Choose a reason for hiding this comment

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

C'est une belle preuve supplémentaire que le secrétariat devrait être un niveau de permission plutôt qu'un service 😬
Très bien vu pour cette jolie amélioration !

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

Successfully merging this pull request may close these issues.

None yet

4 participants