Skip to content

[TECH] Isoler la logique Account Recovery generique pour IAM - #17018

Open
bpetetot wants to merge 9 commits into
devfrom
tech-ddd-recovery-demand
Open

[TECH] Isoler la logique Account Recovery generique pour IAM#17018
bpetetot wants to merge 9 commits into
devfrom
tech-ddd-recovery-demand

Conversation

@bpetetot

@bpetetot bpetetot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

☀️ Problème

Actuellement, la récupération de compte est très lié au sco (et organization-learner), ce qui fait une dépendance forte de prescription/organization-learner dans iam

⛱️ Proposition

Dans un premier temps, isoler la logique propre à iam et la logique propre à prescription/organization-learner.

Dans le futur on pourra imaginer une solution plus globale pour la récupération de compte (hors spécificités sco). C'est pourquoi on souhaite isoler la partie "générique" de la récupération de compte et déplacer la "spécifique" dans prescription/organization-learner afin d'inverser la dépendance.

Dans cette PR on réalise uniquement l'isolation du code. La partie déplacée dans prescription/organization-learner sera faite dans une autre PR.

🧴 Remarques

N/A

🏊 Pour tester

Tester la récupération de comptes.

  1. Sur Pix app, aller sur la page /recuperer-mon-compte
  2. Remplir le formulaire avec :
  • 123456789HG
  • Hermione
  • Granger
  • 05/05/2013 
  1. Envoi d'email pour récupérer son compte
  2. Cliquer sur le lien de l'email pour récupérer le compte (tester avec une mauvaise key également)
  3. Saisir un mot de passe et valider

@bpetetot
bpetetot requested a review from a team as a code owner August 3, 2026 10:39
@bpetetot bpetetot self-assigned this Aug 3, 2026
@pix-bot-github

pix-bot-github commented Aug 3, 2026

Copy link
Copy Markdown

@bpetetot
bpetetot force-pushed the tech-ddd-recovery-demand branch 7 times, most recently from f92e2c2 to a3b300e Compare August 3, 2026 16:17
@bpetetot
bpetetot force-pushed the tech-ddd-recovery-demand branch from a3b300e to 0142bfd Compare August 4, 2026 07:51
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.

2 participants