🚀 Prevent duplicate assignment in rdv edit form
Created by: n-b
refs #2028 (closed)
La bonne façon de faire nécessite d’ajouter une validation d’unicité sur RdvsUsers (et sur AgentsRdvs, tant qu’à faire), d’abord dans ActiveRecord, et surtout en base. Par contre:
- la validation d’unicité dans ActiveRecord ne fonctionne pas de façon satisfaisante, c’est un problème connu de Rails. En gros, il faut se baser sur la contrainte en base et faire du code autour :)
- il faut nettoyer la baddata en base avant de rajouter la contrainte
- mais vu ce bug, on a quelques centaines d’erreurs en base, et ça continue d’arriver; je ne peux donc pas nettoyer à la main puis mettre le fix et la migration en prod en même temps. Cette PR consiste donc à fermer le robinet à baddata.
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