🚀 Prevent RdvsUsers and AgentsRdvs duplicates with DB constraints
Created by: n-b
followup #2034, fixes #2028 (closed)
Concrètement, la validation en ruby ne sert pas à grand chose: Rails ne gère par correctement la validation d’unicité avec les nested_attributes
. L’intérêt de faire passer cette PR en
Vu que la validation est sur une contrainte de DB, s’il y a une erreur, ce sera une 500; cependant côté frontend, en principe, on ne peut pas envoyer ce genre de données. C’est acceptable pour le moment. J’ai bien envie d’améliorer le formulaire de modification (et un jour de création) de Rdv, de toute façon.
Checklist avant review:
-
reparcourir le code rapidement pour voir les problèmes évidents (fichiers touchés inutilement, debug logs qui trainent…). -
Tester la fonctionnalité sur la review app