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

[TECH] Migrer la route GET /api/admin/users/{id} dans src (PIX-15170) #10470

Merged

Conversation

er-lim
Copy link
Contributor

@er-lim er-lim commented Nov 4, 2024

🍂 Problème

Le code de la route GET /api/admin/users/{id} est encore dans lib 🚗

🌰 Proposition

Le migrer dans src/identity-access-management

🎃 Remarques

Le déplacement du usecase get-user-details-for-admin.js impacte aussi l'API permettant d'anonymiser un utilisateur côté Pix Admin

🪵 Pour tester

  • Se connecter à la RA de Pix Admin avec le compte [email protected]
  • Ouvrir la console navigateur => Onglet Network

Test de l'appel côté Pix Admin

Test de non régression de l'anonymisation

  • Aller sur la page de détail d'un utilisateur (ex: https://admin-pr10470.review.pix.fr/users/106951)
  • Cliquer sur le bouton Anonymiser cet utilisateur et confirmer le choix
  • Vérifier que l'appel POST /api/admin/users/${id}/anonymize renvoie bien une 200

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@er-lim er-lim force-pushed the pix-15170-migrate-get-admin-user-details-api branch 2 times, most recently from 530e27f to 020ae0c Compare November 4, 2024 15:57
@er-lim er-lim closed this Nov 4, 2024
@er-lim er-lim reopened this Nov 4, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@er-lim er-lim force-pushed the pix-15170-migrate-get-admin-user-details-api branch from 020ae0c to a6c4668 Compare November 4, 2024 16:10
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

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

Relu et testé ✅

@er-lim par contre, le lien que tu as mis dans tes tests pour l'utilisateur anonymisé ne fonctionne pas (https://admin-pr10470.review.pix.fr/users/106951). Je pense que tu t'es trompé d'id car je ne le retrouve pas. J'ai testé en anonymisant un autre utilisateur - https://admin-pr10470.review.pix.fr/users/106916 - et ça fonctionne bien.

Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay left a comment

Choose a reason for hiding this comment

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

Lu et testé avec succès

@er-lim
Copy link
Contributor Author

er-lim commented Nov 6, 2024

@bpetetot bonne remarque !

J'ai entre temps refait un db:seed sur la review-app pour tester si la commande refonctionne.
Ca a dû modifié les ids des utilisateurs en base 😄

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15170-migrate-get-admin-user-details-api branch 2 times, most recently from 20b44ea to d864e2d Compare November 6, 2024 09:11
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15170-migrate-get-admin-user-details-api branch from d864e2d to 1e47392 Compare November 6, 2024 09:52
@pix-service-auto-merge pix-service-auto-merge merged commit 69aeecf into dev Nov 6, 2024
8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15170-migrate-get-admin-user-details-api branch November 6, 2024 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants