MTES-MCT / Docurba

4 stars 5 forks source link

Modifier nos noms de collaborateurs SuperAdmin #812

Open CeliaVermicelli opened 2 months ago

CeliaVermicelli commented 2 months ago

Problème : les event ajoutés par nos users test apparaissent dans la liste des collaborateurs, exemple : Octavie Test, Célia Vermicelli, Test Test, etc. C'est embêtant pour les event qui doivent être conservés car on les a ajoutés pour aider les user (ex : palier au dépot d'acte qui n'était pas lié à la creation de proc.)

Solution proposée avec Fabien :

On pense que de cette manière : -> ça fera plus professionnel -> sera plus pérenne de ne pas personnifier nos user bizdev -> ça nous permet d'induire le réflexe de contacter l'equipe Docurba

@julienzdocurba qu'en penses-tu ? as-tu d'autres reco pour que "administrateur Docurba" soit bien acceptés par les users ?

Issue concomitante avec la #615 donc pour la MEP du partage de proc. Et sinon, solution de contournement : changer nos noms/prénoms en en BDD en attendant de pouvoir faire mieux.

@Octaviedebs

Pour rappel, voici nos users test : octavie.dbs+test@docurba.beta.gouv.fr celia.vermicelli@docurba.beta.gouv.fr

julienzdocurba commented 2 months ago

On avait dit qu'on cachait les comptes tests et admin de la liste des collaborateurs et que les événements admin auraient pour auteur "Docurba" (on pensait surtout à ca pour pousser manuellement les événements "Docurba"). Par contre je pense qu'on ne devrait pas utiliser les mêmes comptes pour les tests et pour les événements admin autrement on n'aura aucun moyen de savoir lesquels montrer et lesquels cacher.

julienzdocurba commented 2 months ago

On a discuté un peu plus en détail de la question avec @CeliaVermicelli et @Octaviedebs et on a identifié 2 cas:

Et ces 2 actions vont avoir 2 impacts non souhaités:

Pour le cas de la QA, comme pour le cas des événements poussés par Docurba, on ne souhaite pas afficher le compte utilisé dans la liste des collaborateurs (et dans les suggestions de collaborateurs à inviter). On en avait déjà parlé pour le compte admin sur la question des événements automatiques mais je ne sais plus si on avait déjà implémenté ça @UngererFabien .

Pour le cas de la QA, les événements créés devraient en principe avoir toujours été supprimés donc le problème de l'auteur en dessous de l'événement ne se pose pas. Par contre pour le cas des événements poussés par Docurba, on s'est dit qu'on utiliserait tous le compte "admin" pour ca, et que l'auteur serait donc simplement "Docurba" (dites moi si je me trompes @Octaviedebs et @CeliaVermicelli ). Encore une fois, c'est le système dont on avait parlé avec Hermance pour les événements automatiques Docurba.

Le point qu'on n'a jamais évoqué c'est ce qu'on fait avec les événements existants. Comme tous les événements de QA ont été supprimés, ce sont tous des événements du cas 2 et on souhaiterait donc idéalement que l'auteur soit "Docurba" mais que le compte n'apparaisse pas dans la liste des collaborateurs. Et même si les événements de QA ont été supprimé, on voudrait que les comptes qui les ont ajoutés n'apparaissent pas dans la liste des collaborateurs. Est-ce qu'il faudrait effectuer une migration? Ou bien juste changer le nom d'utilisateur des comptes de Célia et Octavie pour "Docurba" en base de données directement?