-
Notifications
You must be signed in to change notification settings - Fork 58
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
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
e24b081
to
45d1eed
Compare
api/src/identity-access-management/domain/usecases/authenticate-oidc-user.usecase.js
Outdated
Show resolved
Hide resolved
mon-pix/app/controllers/authentication/login-or-register-oidc.js
Outdated
Show resolved
Hide resolved
api/src/identity-access-management/domain/usecases/authenticate-oidc-user.usecase.js
Outdated
Show resolved
Hide resolved
api/src/identity-access-management/domain/usecases/authenticate-oidc-user.usecase.js
Outdated
Show resolved
Hide resolved
mon-pix/app/components/authentication/login-or-register-oidc.js
Outdated
Show resolved
Hide resolved
api/src/identity-access-management/application/oidc-provider/oidc-provider.controller.js
Outdated
Show resolved
Hide resolved
87fd92c
to
edcdab8
Compare
edcdab8
to
44ded73
Compare
api/src/identity-access-management/domain/usecases/authenticate-oidc-user.usecase.js
Show resolved
Hide resolved
171fb51
to
9913d31
Compare
mon-pix/app/components/authentication/login-or-register-oidc.hbs
Outdated
Show resolved
Hide resolved
e493d3b
to
db33620
Compare
mon-pix/app/controllers/authentication/login-or-register-oidc.js
Outdated
Show resolved
Hide resolved
mon-pix/app/controllers/authentication/login-or-register-oidc.js
Outdated
Show resolved
Hide resolved
e086f4f
to
780056c
Compare
There was a problem hiding this 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.
There was a problem hiding this 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 🦊
780056c
to
773ff8c
Compare
773ff8c
to
b48f4dc
Compare
🌸 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
npm run db:reset
).