Skip to content

fix/issues

Marlène SIMONDANT requested to merge fix/issues into V2.0

What does this MR do and why?

Retours review sprint 3 :

Annuaire

  • US annuaire : Pagination texte à changer par “20 utilisateurs sur 23”.
  • Menu pour les filtres, il faudrait férrer à gauche comme sur la carto.
  • Lorsque le résultat de la recherche est ≤ 1, il faut mettre le pluriel entre parenthèse -> utilisateur(s)
  • () Le bouton “voir plus” ne charge pas toujours correctement le reste de la liste utilisateurs
  • Inverser l’odre de fonction et employeur car l’employeur est à droite dans la liste de resultats
  • () Lorsqu’on clique sur un profil avec un filtre actif, le volet de profil ne charge pas
  • Lorsqu’on clique sur un profil avec un filtre actif, le volet de profil se charge maintenant, mais quand on revient les filtres restent affichés mais il y a 0 utilisateurs trouvés
  • Dans la page profil depuis l’annuaire → le bouton “Voir ma structure” devrait être visible en page public, comme sur le comportement de mon profil (ok en local, pb de déploiement, corrigé par le fix(cicd): update V2.0 job de HugoS)

Profil

  • Lorsque je sauvegarde ma description, ça dit bien “votre description a été mis à jour” alors que les autres onglet sont “informations mise à jour” -> mettre informations mise à jour pour tous
  • Mail de relance structure à mettre à jour → typo manque accent sur le a et mettre “Inclusion” au lieu de “Médiation”
  • () Lorsque je valide une information, le bouton “Valider” devrait fermer la fenetre et me faire revenir à “Modifier la structure”
  • Dans “Gérer les membres”, le nom de la structure n'est pas en rouge et fait un retour à la ligne, prendre plutôt le même comportement que “Modifier structure”
  • () Bouton Retour depuis gérer les membres KO lorsqu’on a ajouté des membres ou exclu, si on ne fait aucune action ça retourne bien comme il faut.

- [ ] Ajouter un membre déjà existant dans la base n’envoie pas de mail ? () US À FAIRE

Général

- [ ] Il manque le loader sur la carto lorsqu’on charge les structures, en plus ça met du temps US À FAIRE

  • Clean la base de res'in dev car trop de comportement incohérents

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