Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BO] Tagger les mails des comptes et partenaires archivés pour éviter les doublons #2366

Merged
merged 4 commits into from
Apr 4, 2024

Conversation

hmeneuvrier
Copy link
Collaborator

Ticket

#2035

Description

Lorsqu'on archive un user ou un partenaire, on tag le champ email pour qu'il n'y ait plus de doublon .archived@yyyymmddhhmm

Changements apportés

  • ajout d'une page des partenaires archivés
  • à l'archivage d'un utilisateur et d'un partner on tag le mail avec .archived@yyyymmddhhmm
  • au désarchivage, on vérifie que le mail n'est pas utilisé ailleurs, et on affiche une alerte si besoin (et on retire le tag si ok)
  • à l'affichage on masque le tag
  • on fait une commande pour taguer les mails de tous les utilisateurs et partenaires archivés

Pré-requis

Tests

  • Lancer la migration et vérifier en base que les utilisateurs et partenaires archivés ont bien le tag
  • Aller voir dans le BO que le tag n'apparait pas dans les page de comptes archivés et partenaires archivés
  • Archiver un utilisateur, et vérifier qu'il apparait bien dans les comptes archivés
  • Archiver un partenaire, et vérifier qu'il apparait bien dans les partenaires archivés, et que ses utilisateurs apparaissent dans les comptes archivés
  • Désarchiver un utilisateur, vérifier qu'on ne voit pas le tag, et qu'il est bien désarchiver (vérifier le mail en base)
  • Idem pour un partenaire
  • Aller changer un utilisateur en base pour lui mettre l'e-mail d'un utilisateur archivé (sans le tag évidemment)
  • Essayer d'aller désarchiver l'utilisateur qui a le même mail et vérifier qu'on a bien une alerte

Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test pagination de la nouvelle page Partenaires "Partenaires archivés" : KO

src/Controller/Back/BackArchivedPartnerController.php Outdated Show resolved Hide resolved
templates/back/nav_bo.html.twig Outdated Show resolved Hide resolved
@hmeneuvrier hmeneuvrier force-pushed the bugfix/2035-tag-mail-user-partner-archived branch from d497a3e to f48c1b1 Compare March 29, 2024 09:33
Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je suis tombé sur un petit bug en faisant mes tests (voir commentaire de code)
Tout le reste est OK

src/Twig/AppExtension.php Outdated Show resolved Hide resolved
@hmeneuvrier hmeneuvrier force-pushed the bugfix/2035-tag-mail-user-partner-archived branch from 2e5701e to a9a6bdb Compare April 2, 2024 07:51
@hmeneuvrier hmeneuvrier force-pushed the bugfix/2035-tag-mail-user-partner-archived branch from a9a6bdb to 3ce1b3c Compare April 4, 2024 08:37
Copy link

sonarcloud bot commented Apr 4, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
19.7% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

Copy link
Collaborator

@sfinx13 sfinx13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lecture et test OK

@numew numew merged commit 1cf68ea into develop Apr 4, 2024
2 of 3 checks passed
@hmeneuvrier hmeneuvrier deleted the bugfix/2035-tag-mail-user-partner-archived branch April 11, 2024 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants