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

Anonymisation des comptes agents expiré #2594

Merged
merged 3 commits into from
May 31, 2024

Conversation

numew
Copy link
Collaborator

@numew numew commented May 28, 2024

Ticket

#2429
#2584

Description

  • Ajout de la commande app:anonymize-expired-account qui anonymize les comptes agent sans activité depuis plus de deux ans à condition qu'il soit archivé (ce qui est fait automatiquement sans réponse de leur part via la commande app:notify-and-archive-inactive-accounts
  • Correction du process de désarchivation d'un compte afin que l'utilisateur en cours de désarchivage ne soit pas bloqué par son propre e-mail (cas de soumission du formulaire sur un utilisateur sans partenaire/territoire non archivé)

Pré-requis

make load-migrations

Tests

  • Tester la commande app:anonymize-expired-account et verifier que les comptes anonymisés ne sont affiché et pris en compte nulle part (comptes archivés, inactifs, expirés)
  • S'assurer que l'on anonymise uniquement les agent inactif depuis plus de deux ans qui sont déja archivé
  • S'assurer que le système de désarchivage fonctionne correctement et qu'il es impossible de désarchiver un user anonymisé

src/Entity/User.php Outdated Show resolved Hide resolved
tests/Unit/Entity/UserTest.php Outdated Show resolved Hide resolved
src/Controller/Back/PartnerController.php Outdated Show resolved Hide resolved
@hmeneuvrier
Copy link
Collaborator

Pour les tests c'est OK

Copy link
Collaborator

@emilschn emilschn left a comment

Choose a reason for hiding this comment

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

Pour la lecture, ok avec les coms d'Hélène. Sinon RAS.
Tests ok !

Copy link

sonarcloud bot commented May 30, 2024

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarCloud

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

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

Ok pour moi, tests et relecture

@hmeneuvrier hmeneuvrier merged commit 3910c4e into develop May 31, 2024
2 of 3 checks passed
@numew numew mentioned this pull request May 31, 2024
@hmeneuvrier hmeneuvrier deleted the feature/2429-anonymization-agent-expire branch June 11, 2024 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants