enhancement/add_wrong_organization_error_message

[qblanc]

Dans cette PR, j'ai

• ajouté la prise en compte spécifique du cas où l'agent tentant de créer les usagers n'appartient pas à l'organisation Ardennes (bug rencontré par Claire),
• séparé l'appel de getUser().
• simplifié (ou plutôt, péniblement essayé de simplifier) l'écriture des différents if pour la prise en charge des erreurs, avec des ? puis finalement des shift()

Pour la prise en charge de l'erreur :

• J'ai envisagé dans un premier temps de le faire au moment du login, mais les infos sur l'agent renvoyées dans le body de la response comprennent le service, mais pas l'organisation (et les infos renvoyés sont générées automatiquement par devise_token_auth, une customization aurait pris trop de temps pour l'utilité).
• J'ai ensuite pensé lancer un get de user au moment du login, mais ce get n'est possible que sur les users (usagers) et pas sur les agents. C'est à ce moment-là que j'ai séparé la méthode getUser() de la méthode checkUserInvitationStatus() ; le faire n'a plus d'utilité directe au final, mais cela pourra reservir par la suite, j'ai donc décider de garder ce changement.
• J'ai finalement opté pour la prise en charge de cette erreur au moment de la tentative de création de compte.

[qblanc]

J'ai pris en compte la plupart de tes remarques. Les syntaxes du style else if (result?.errors.email && result.errors.email[0] === "taken") sont pensées pour raccourcir else if (result.errors && result.errors.email && result.errors.email[0] === "taken") ; est-ce que c'est correct ?

[aminedhobb]

J'ai pris en compte la plupart de tes remarques. Les syntaxes du style else if (result?.errors.email && result.errors.email[0] === "taken") sont pensées pour raccourcir else if (result.errors && result.errors.email && result.errors.email[0] === "taken") ; est-ce que c'est correct ?

@qblanc result?.errors vérifie que result n'est pas nulle (ce qui n'est à priori jamais le cas). Je pense que la deuxième manière d'écrire est plus cohérente.

Ce sont toujours des 422 qui sont renvoyées il me semble, et cela concerne des cas assez différents, ce n'est pas mieux de personnaliser le message d'erreur renvoyé à l'utilisateur ?

Oui pour personnaliser le message, mais du coup je voulais juste remplacer la condition message === "forbidden" par reponse.code === 403.