Skip to content

[agent] create new organisation autonomously for admins

Created by: adipasquale

https://trello.com/c/axi8ALDy/974-self-service-permettre-la-cr%C3%A9ation-dune-nouvelle-organisation-par-un-agent-d%C3%A9j%C3%A0-existant-dans-une-autre-organisation

Screenshots

stitched_2020_08_27-10_57_55

Changements

  • seuls les agents admins voient le lien et peuvent créer une autre orga
  • La page liste des organisations devient accessible meme quand il n'y a qu'une orga (avant ca redirigeait automatiquement vers l'orga). La redirection auto continue a exister, mais uniquement apres le signin de l'agent et/ou l'acces a la racine.

Discussion route espace agents

Aujourd'hui il n'y a pas vraiment de préfixe de route pour l'espace agent. Il y a un préfixe pour l'espace "superadmin" qui est /admin.

Dans l'espace agent la plupart des routes commencent par /organisations/:id mais pas toutes :

  • /organisations liste les orgas
  • departements/75 affiche l'interface de secto

ca ne genait pas jusqu'ici de ne pas avoir de préfixe mais pour cette feature j'ai un conflit. Je dois créer un nouveau formulaire de création d'organisations pour les agents admin authentifiés, et il existe deja une route /organisations/new publique pour l'autonomisation. Les deux formulaires sont assez differents, et l'action de creation dans les deux cas est completement differente. J'ai essayé d'utiliser la route existante et de brancher un peu partout pour afficher l'un ou l'autre formulaire, et exercer l'une ou l'autre action mais ce n'est vraiment pas beau ni facile a faire.

J'ai donc adopté une deuxieme solution : résoudre le conflit de routes en préfixant les routes d'orgas de l'espace agent:

  • /organisations devient /backoffice/organisations
  • et la nouvelle page peut devenir /backoffice/organisations/new sans conflit avec l'autre

je n'ai pour l'instant pas etendu ce prefixe a tout l'espace agent mais je pense qu'il faudrait le faire si on choisit d'aller dans cette direction.

Avantages du prefixe :

  • (je crois) plus correct semantiquement 1 URL = 1 vue de ressource, sans trop de variabilité.
  • evite des conflits futurs, par exemple /rdvs/123 pour les usagers versus /backoffice/rdvs/33
  • clarifie le nom du module agents en backoffice par ex (et donc du repertoire associé). ca l'alignerait aussi avec les routes.

Inconvénients :

  • un peu de boulot pour migrer, introduction d'erreurs potentielles
  • verbeux en prefixes partout dans les path helpers
  • veut-on gerer la redirection des anciennes routes pour les bookmarks d'agents eventuels? je dirais que non
  • je ne trouve pas de nom genial, backoffice est vraiment naze, c'est plus un placeholder qu'une vraie proposition

Merge request reports