-
Notifications
You must be signed in to change notification settings - Fork 86
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
Dossier state change logs #3060
Dossier state change logs #3060
Conversation
76dfe02
to
ded2575
Compare
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.
Sur la technique, ça me semble raisonnable (avec une question sur l'usage d'un Service).
Sur le plan métier, je me sens un peu démuni pour évaluer le besoin. Quelle est la demande des utilisateurs ? Est-ce que c'est pour les instructeurs, les admins ? Quels sont les cas d'usages ? (On peut créer une issue pour en parler sans polluer la PR, éventuellement)
Il y a plusieurs motivations. Mais de manière générale, je trouve que ça facilitera le debug et la traçabilité dans le système. Il y a aussi le plan d'exposer une API de de changement de statut du dossier et dans ce cas je veux vraiment imposer qu'un instructeur soit systématiquement passé à l’API a fin de garder une trace chez nous de qui fait quoi. Aujourd'hui, le seul moyen de savoir quel est l'instructeur qui a fait une opération c'est d’aller fouiller dans les log HTTP :( |
e546253
to
ef9ae20
Compare
802d177
to
9b3a449
Compare
9b3a449
to
e0eae6b
Compare
@gregoirenovel up avec la découpe comme tu l'as demandé |
e0eae6b
to
0f0ca2e
Compare
No description provided.