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

[FEATURE] Modifier l'interface de la double mire SSO pour inclure toutes les données récupérées des utilisateurs (PIX-16303) #11861

Merged

Conversation

P-Jeremy
Copy link
Contributor

@P-Jeremy P-Jeremy commented Mar 27, 2025

🌸 Problème

Actuellement, sur la page de la double mire SSO, la liste des informations récupérées auprès d'un SSO OIDC sont écrites en dur au lieu d'être dynamiques en fonction de ce qui est récupéré effectivement auprès de chaque SSO.

🌳 Proposition

Rendre générique la récupération et l'affichage des userClaims.

🐝 Remarques

Les informations sont désormais stockées dans le session storage et non plus passées en queryParams

🤧 Pour tester

  • Sur Mon Pix
    • Se connecter avec le sso fwb
    • Constater que le numéro d'employé et la population s'affichent dans la partie "Je n’ai pas de compte Pix"

image

  • Se connecter avec le sso pays de la Loire
  • Constater que seuls le Prénom et Nom s'affichent dans la partie "Je n’ai pas de compte Pix"

image

  • Supprimer les "userClaims" dans "oidcUserAuthentication" du session storage
  • rafraichir la page
  • Constater le message d'erreur

Erreur : Nous n’avons pas pu récupérer vos informations d’identité auprès du service utilisé. Nous vous invitons à contacter le support informatique de cette organisation.

  • Sur Pix Admin : vérifier qu'il est toujours possible de se connecter par SSO (bien repartir d'un situation avec BDD dans un état initial après un npm run db:reset).

@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 :

@P-Jeremy P-Jeremy force-pushed the pix-16303-display-user-infos-on-sso-register-form branch 4 times, most recently from e24b081 to 45d1eed Compare March 27, 2025 12:17
@P-Jeremy P-Jeremy force-pushed the pix-16303-display-user-infos-on-sso-register-form branch 6 times, most recently from 87fd92c to edcdab8 Compare March 31, 2025 13:53
@P-Jeremy P-Jeremy marked this pull request as ready for review March 31, 2025 14:08
@P-Jeremy P-Jeremy requested a review from a team as a code owner March 31, 2025 14:08
@P-Jeremy P-Jeremy added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally and removed Development in progress labels Mar 31, 2025
@P-Jeremy P-Jeremy force-pushed the pix-16303-display-user-infos-on-sso-register-form branch from edcdab8 to 44ded73 Compare March 31, 2025 15:10
@lego-technix lego-technix self-requested a review April 1, 2025 07:48
@P-Jeremy P-Jeremy added Development in progress and removed 👀 Func Review Needed Need PO validation for this functionally labels Apr 1, 2025
@P-Jeremy P-Jeremy force-pushed the pix-16303-display-user-infos-on-sso-register-form branch from 171fb51 to 9913d31 Compare April 1, 2025 12:55
@P-Jeremy P-Jeremy force-pushed the pix-16303-display-user-infos-on-sso-register-form branch 4 times, most recently from e493d3b to db33620 Compare April 2, 2025 08:07
@P-Jeremy P-Jeremy added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally and removed Development in progress labels Apr 2, 2025
lego-technix

This comment was marked as resolved.

@P-Jeremy P-Jeremy force-pushed the pix-16303-display-user-infos-on-sso-register-form branch 2 times, most recently from e086f4f to 780056c Compare April 2, 2025 12:39
Copy link
Contributor

@lego-technix lego-technix 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 revue de code.

Copy link
Contributor

@lego-technix lego-technix 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é fonctionnellement avec succès avec Firefox 🦊

@P-Jeremy P-Jeremy added Tech Review OK 🚀 Ready to Merge Func Review OK PO validated functionally the PR and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Apr 3, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16303-display-user-infos-on-sso-register-form branch from 780056c to 773ff8c Compare April 3, 2025 11:56
@yaf yaf force-pushed the pix-16303-display-user-infos-on-sso-register-form branch from 773ff8c to b48f4dc Compare April 3, 2025 14:10
@pix-service-auto-merge pix-service-auto-merge merged commit cab0b4f into dev Apr 3, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16303-display-user-infos-on-sso-register-form branch April 3, 2025 14:16
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.

6 participants