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/oidc/users vers src/identity-access-management (PIX-12503) #8906

Merged
merged 9 commits into from
May 17, 2024

Conversation

igorissen
Copy link
Contributor

@igorissen igorissen commented May 14, 2024

🦄 Problème

La route POST /api/oidc/users est encore dans le dossier lib.

🤖 Proposition

Faire la migration vers son bounded context identity-access-management.

🌈 Remarques

Les derniers commits de cette PR créent également des sous-répertoires dans src/identity-access-management/application/ pour classer encore plus les routes par thématique.

💯 Pour tester

  • Faire le test de connexion, ainsi que la création d'un nouveau compte sur tous les fournisseurs d'identité.
  • Se reconnecter avec le même compte pour chacun des fournisseurs d'identité

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

@igorissen igorissen changed the title Migrer la route POST /api/oidc/users vers src/identity-access-management [TECH] Migrer la route POST /api/oidc/users vers src/identity-access-management (PIX-12503) May 14, 2024
@lego-technix lego-technix self-requested a review May 14, 2024 09:31
@igorissen igorissen force-pushed the bc-move-oidc-create-user-route-to-iam branch from 698f811 to e6e19b4 Compare May 14, 2024 10:04
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

Copy link
Contributor

@er-lim er-lim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test en local sur firefox ✅

to identity-access-management bounded context
to identity-access-management bounded context
to identity-access-management bounded context
to identity-access-management bounded context
to identity-access-management bounded context
@pix-service-auto-merge pix-service-auto-merge force-pushed the bc-move-oidc-create-user-route-to-iam branch from 8ef7281 to 3ea8589 Compare May 17, 2024 07:41
@pix-service-auto-merge pix-service-auto-merge merged commit 58612cd into dev May 17, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the bc-move-oidc-create-user-route-to-iam branch May 17, 2024 08:06
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.

None yet

5 participants