Skip to content

🚀 Prevent RdvsUsers and AgentsRdvs duplicates with DB constraints

Vincent Agnano requested to merge prevent-user-duplicates-in-rdv into production

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 🚀 est surtout de faire passer la migration (et de s’assurer qu’elle n’échoue pas) indépendamment du reste de la recette.

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

Merge request reports