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] Faciliter les changements de noms des fournisseurs d'identité OIDC (pix-15616) #10805

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

EmmanuelleBonnemay
Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay commented Dec 13, 2024

🎄 Problème

Le changement de nom de la CNAV a généré deux incohérences dans Pix Admin :

  • Le nom qui apparaît dans la modale de confirmation de suppression de méthode d'authentification reste "CNAV"
  • Le nom qui apparaît dans la pop up d'erreur lors de la ré-assignation d'une méthode de connexion SSO reste "CNAV"

🎁 Proposition

Profiter de ces corrections pour faciliter les changements de noms à venir

🧦 Remarques

RAS

🎅 Pour tester (en local)

  • Sur Pix App : Réconcilier les deux comptes de tests CNAV avec des utilisateurs ayant une méthode de connexion PIX (par exemple, James Paledroits et Marc-Alex Terrieur). Il faut le faire sur 2 navigateurs différents, à cause des spécificités de déconnexion de la CNAV.
  • Vérifier le déplacement de la méthode de connexion de la CNAV de James vers Marc-Alex, constater que le message d'erreur affiche bien Caisse Nationale d’Assurance Retraite et CARSAT
  • Vérifier que les modales de suppression de méthode d'authentifications affichent le noms correct Caisse Nationale d’Assurance Retraite et CARSAT pour la CNAV

Et ensuite, non régression en général sur toutes les méthodes de connexion : Vérifier la suppression et /ou le déplacement d'autres moyens de connexion : le GAR, un fournisseur d'identité OIDC, une autre méthode d'authentification (vérifier dans ce dernier cas que la donnée s'actualise bien dans les 'informations de l'utilisateur' ).

@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

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

Profiter de cette PR pour, dans des commits supplémentaires, supprimer les dernières mentions de CNAV dans le code qui n'ont donc plus lieu d'être :

  • admin/tests/unit/components/users/user-detail-personal-information/authentication-method-test.js
  • docker/sample.env (et des autres OIDC providers présents aussi dans ce fichier)

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

@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-15616-remove-obsolete-identity-provider-name branch from 35722d6 to 19cf5ba Compare December 16, 2024 12:44
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.

4 participants