Skip to content

Feat/us141 structure exclude member

Etienne LOUPIAS requested to merge feat/US141-structure-revoke-member into V2.0

What does this MR do and why?

Cette MR permet d'exclure un membre qui a rejoint une structure (le 1er user qui rejoint une structure doit être validé par l'admin résin).

Cette branche est basée sur la branche de l'US140 de Rémi.

  • Sur réception du mail indiquant qu'une personne a rejoint la structure, un lien est proposé pour signaler l'erreur

  • Ce lien renvoi vers l'écran de connexion puis une fois l'utilisateur connecté, affiche une pop-up. 2 cas sont présents : le membre a déjà été exclus de la structure, ou le membre est toujours présent dans la structure

  • Quand un user clique sur "Je travaille ici" sur la page d'une structure

    • s'il est le premier à vouloir rejoindre la structure (la revendiquer), la structure va en pendingStructures et un email est envoyé à l'admin Résin pour validation (comme dans la version v1 de Resin)
    • s'il y a déjà des membres dans la structure, il est directement affecté à la structure (la structure n'est plus mise dans ses pendingStructures), le mail pour signaler une erreur d'appartenance et envoyée aux membres de la structure, et une fenetre d'information est affichée au user (US140 de Rémi)

Screenshots or screen recordings

Exclure Déjà exclu

How to set up and validate locally (or on alpha)

  • Prendre une structure de test sans membre (en créer une nouvelle et/ou supprimer ses membres en base)
  • Avec un compte A non admin resin (dont on a accès à la boite mail), sur la page de la structure, cliquer sur "Je travaille ici"
  • Avec un compte admin de Résin, valider la revendication de la structure
  • Avec un autre compte B non admin resin, sur la page de la structure, cliquer sur "Je travaille ici"
  • le compte B devient membre de la structure (il a le bouton "Modifier cette structure")
  • le mail pour signaler une erreur d'appartenance est envoyé au compte A
  • dans le mail cliquer sur le lien puis sur le bouton "Exclure"
  • le compte B n'est plus membre de la structure
  • dans le mail recliquer sur le lien pour signaler une erreur d'appartenance, la popup informant que le membre a déjà été exclu de la structure s'affiche

MR acceptance checklist

To be completed by the chosen reviewer.

Quality

  • Confirmed
  1. For the code that this change impacts, I believe that the automated tests validate functionality that is highly important to users. If the existing automated tests do not cover this functionality, I have added the necessary additional tests or I have added an issue to describe the automation testing gap and linked it to this MR.
  2. I have made sure that the sonar quality coverage is up to standards.
  3. I have considered the impact of this change on the front-end, back-end, and database portions of the system where appropriate and applied.
  4. I have tested this MR in all supported browsers or determined that this testing is not needed.
  5. I have confirmed that this change is backwards compatible across updates (migrate up needs a migrate down), or I have decided that this does not apply.

Performance, reliability and availability

  • Confirmed
  1. I am confident that this MR does not harm performance, or I have asked a reviewer to help assess the performance impact.
  2. I have considered the scalability risk based on future predicted growth.

Documentation

  • Confirmed
  1. I have prepared a squash commit to feed the changelog linked to the current milestone.
  2. I have added/updated documentation (also updated if the changes feature a deprecation) or I have decided that documentation changes are not needed for this MR.

Security

  • Confirmed
  1. I have confirmed that if this MR does not contains any sensitive informations hidden in the changes.

Deployment

  • Confirmed
  1. When featured on a self-data project release, i have made sure my app version in the manifest and package.json is incremented and any relative changes to the permissions are clearly written and transmitted to Cozy.
Edited by Etienne LOUPIAS

Merge request reports