Skip to content

Afficher un message quand on invite un agent sur un autre service

Vincent Agnano requested to merge warn-invitations-different-service into master

Created by: n-b

close #1281 (closed)

  • J’ai aussi mis du refactor de la gestion d’erreur, mais peut-être que j’utilise trop les trucs profonds de Rails.
  • J’ai le sentiment que la liste d’agents et la liste d’invitations devraient être la même page, ou le même contrôleur. Je n’ai pas fini de fouiller, mais est-ce que ça n’était pas le cas à une époque? Il y a des indices dans ce sens.
    • EDIT: c’était #1099. Le code des deux tables est maintenant très similaires, je n’ai pas tout factorisé, mais on pourrait sans doute. Ça devient hors-scope pour cette PR.
  • Il y a un truc qui gratte à la validation des utilisateurs invités (non acceptés) qu’on invite aussi dans une deuxième organisation: on les modifie, mais en principe, leur first_name et last_name sont alors requis. J’avais déjà touché à ça dans une autre PR.
    • EDIT: C’est corrigé: j’utilise un contexte de validation spécifique pour passer outre ces validations.
  • Tests de non-régression à écrire:
    • La validation custom du modèle AgentRole.
    • L’affichage du warning de différent service dans invitations_devise_controller.

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