Skip to content

Merge Agents::RelativesController into Agents::UsersController

Created by: adipasquale

️ dupliquée depuis #662 pour forcer la création d'une nouvelle review app

Refacto préliminaire pour #643 :

Fusion du controller Agents::RelativesController dans Agents::UsersController

La PR est un peu grosse, les commits sont lisibles un par un mais pas tout à fait bien isolés (on ne pourrait pas les déployer un par un).

Motivations :

  • il y a beaucoup de code redondant entre les deux (cette PR fait -110l.)
  • c'est dangereux le comportement et les validations sont incohérents alors que le modèle sous-jacent est le même (ex : le duplicate finder pas appelé d'un côté)
  • l'évolution du produit justifie de rapprocher les formulaires et de permettre la création d'un proche sans passer par la fiche usager responsable : cette PR rendra ça plus facile

Explications :

  • j'ai gardé deux fichiers de vues séparées pour le show. C'est discutable on pourrait aussi assez facilement en avoir un seul avec des branchements. Avec deux fichiers de vues séparées on est moins DRY mais peut-être un peu plus lisible et évolutif. Je n'ai pas un avis très figé
  • j'ai enlevé les réponses right bar du UsersController , elles ne sont plus utilisées

Merge request reports