Skip to content

🚀 Prevent duplicate assignment in rdv edit form

Vincent Agnano requested to merge fix-duplicate-rdv-users into production

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

Merge request reports