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 POST /api/expired-password-updates (PIX-16367) #11305

Conversation

EmmanuelleBonnemay
Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay commented Feb 3, 2025

🥞 Problème

La route POST /api/expired-password-updates est encore dans /lib

🥓 Proposition

Migrer cette route dans /src/identity-access-management

🧃 Remarques

Des tests d'acceptance de la route GET /api/password-reset-demands/{temporaryKey} ont été migrés également, car cette route est déjà dans /src

😋 Pour tester

  • Aller sur Pix Orga et se connecter avec le compte allorga@example.net
  • Aller sur Élèves
  • Choisir un élève possédant une méthode identifiant et/ou adresse e-mail (par exemple Eliza) et cliquer sur le bouton d'action tout à droite "Gérer le compte"
  • Cliquer sur le bouton "Réinitialiser le mot de passe" et entrer un nouveau mot de passe
  • récupérer l'identifiant et nouveau mot de passe de l'élève et aller sur Mon Pix
  • Sur la page connexion, renseigner les deux informations et valider
  • Constater que la route POST /api/expired-password-updates a bien été appelée et qu'elle a renvoyé une 201 avec une réponse de type
{
	"data": {
		"type": "reset-expired-password-demands",
		"attributes": {
			"login": "{identifiant de l'élève}"
		}
	}
}
  • constater que l'on est bien connecté

@EmmanuelleBonnemay EmmanuelleBonnemay requested a review from a team as a code owner February 3, 2025 14:32
@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 :

@EmmanuelleBonnemay EmmanuelleBonnemay self-assigned this Feb 3, 2025
@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-16367-migrate-post-api-expired-password-updates branch from 2745338 to 0e94e51 Compare February 3, 2025 15:21
@EmmanuelleBonnemay EmmanuelleBonnemay added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Feb 3, 2025
@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-16367-migrate-post-api-expired-password-updates branch 3 times, most recently from 779f5fc to 14c720f Compare February 3, 2025 17:01
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.

Testé et revu ✅

@yaf yaf added this to the Bounded context migration milestone Feb 5, 2025
@Libouk
Copy link
Member

Libouk commented Feb 7, 2025

Ce n'est pas directement lié mais le bouton de copie du nouveau MDP n'est pas visible à côté du champ. On crée un nouveau ticket ?
image

@Libouk
Copy link
Member

Libouk commented Feb 7, 2025

Visible au survol, mais décalé
image

@Libouk
Copy link
Member

Libouk commented Feb 7, 2025

Quand j'ai réinitialisé le MDP depuis Orga puis que j'essaye de me connecter sur Pix App je reçois d'abord une 401.
image

@Libouk Libouk self-requested a review February 7, 2025 09:39
@theotime2005 theotime2005 requested review from theotime2005 and removed request for Libouk February 7, 2025 09:39
@EmmanuelleBonnemay
Copy link
Contributor Author

EmmanuelleBonnemay commented Feb 7, 2025

ça me semble normal car l'accès est refusé tant que le mot de passe n'a pas été réinitialisé

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16367-migrate-post-api-expired-password-updates branch from 14c720f to e24cecd Compare February 7, 2025 13:48
@pix-service-auto-merge pix-service-auto-merge merged commit d7ee5fa into dev Feb 7, 2025
9 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16367-migrate-post-api-expired-password-updates branch February 7, 2025 13:55
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.

7 participants