fix http500 when identifiant is missing #133

Closed
opened 2026-05-21 17:57:50 +02:00 by unurled · 3 comments
Owner

Pull Request synced from GitHub: https://github.com/flambeaux-org/Sacadoc/pull/129
State: merged | Base: mainHead: 500/reset-password
Author: @mpeterschmitt | Created: 2026-04-13


Si tu peux check par rapport au mails qu’on reçoit, j’ai l’impression c’est des bots

> **Pull Request synced from GitHub:** https://github.com/flambeaux-org/Sacadoc/pull/129 > **State:** merged | **Base:** `main` ← **Head:** `500/reset-password` > **Author:** @mpeterschmitt | **Created:** 2026-04-13 --- Si tu peux check par rapport au mails qu’on reçoit, j’ai l’impression c’est des bots
unurled 2026-05-21 17:57:50 +02:00
Author
Owner

@mpeterschmitt commented on GitHub:

@logut Peux tu faire ce que tu as prévu ? j'avoue ne pas être sir de la façon merci !

<!-- gh-comment-id:4319425614 --> **@mpeterschmitt** commented on GitHub: @logut Peux tu faire ce que tu as prévu ? j'avoue ne pas être sir de la façon merci !
Author
Owner

@logut commented on GitHub:

C'est fait :)
En regardant plus, je suis toujours pas sûr qu'on a besoin de demander le mail + l'identifiant, mais faudrait vérifier qu'on a pas de mail en double en bdd entre les utilisateurs et les individus avant d'arrêter de le demander. Ni qu'on ait d'invidu rattaché à 2 familles

<!-- gh-comment-id:4320343536 --> **@logut** commented on GitHub: C'est fait :) En regardant plus, je suis toujours pas sûr qu'on a besoin de demander le mail + l'identifiant, mais faudrait vérifier qu'on a pas de mail en double en bdd entre les utilisateurs et les individus avant d'arrêter de le demander. Ni qu'on ait d'invidu rattaché à 2 familles
Author
Owner

(inline review comment on 500/reset-password branch)
@logut commented on GitHub:

Si on ne sur-définit pas le champ email, il devrait hériter du from de base, qui le défini en tant qu'EmailField et a donc la bonne validation directement, ça permet de retirer complètement la fonction clean() du form

<!-- gh-comment-id:3076111516 --> *(inline review comment on `500/reset-password` branch)* **@logut** commented on GitHub: Si on ne sur-définit pas le champ email, il devrait hériter du from de base, qui le défini en tant qu'EmailField et a donc la bonne validation directement, ça permet de retirer complètement la fonction clean() du form
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
mirror/sacadoc#133
No description provided.