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] Migration de la route POST /api/admin/certification-centers/{certificationCenterId}/certification-center-memberships (PIX-16451) #11357

Conversation

P-Jeremy
Copy link
Contributor

@P-Jeremy P-Jeremy commented Feb 6, 2025

🥞 Problème

La route POST /api/admin/certification-centers/{certificationCenterId}/certification-center-memberships est encore dans /lib

🥓 Proposition

La migrer vers src/team

🧃 Remarques

😋 Pour tester

  • Sur Pix-Admin
  • Depuis la page de détail d'un centre de certification
  • Tenter j'aouter un membre par email via l'input
  • Constater le succès de l'opération

@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 added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally and removed Development in progress labels Feb 6, 2025
@P-Jeremy P-Jeremy force-pushed the pix-16451-migrate-certification-center-membership-creation-route-from-admin branch from 66b399a to 3f475a4 Compare February 7, 2025 08:58
@EmmanuelleBonnemay EmmanuelleBonnemay self-requested a review February 7, 2025 09:40
@Libouk
Copy link
Member

Libouk commented Feb 7, 2025

Membre ajouté, PR validée, Jeremy Pluqué

@P-Jeremy P-Jeremy force-pushed the pix-16451-migrate-certification-center-membership-creation-route-from-admin branch from 3f475a4 to 63478f2 Compare February 7, 2025 10:32
Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay 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é avec succès

@P-Jeremy P-Jeremy added Tech Review OK Func Review OK PO validated functionally the PR 🚀 Ready to Merge and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Feb 7, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16451-migrate-certification-center-membership-creation-route-from-admin branch from 63478f2 to f8eb01a Compare February 7, 2025 13:29
@pix-service-auto-merge pix-service-auto-merge merged commit 3effeeb into dev Feb 7, 2025
9 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16451-migrate-certification-center-membership-creation-route-from-admin branch February 7, 2025 13:41
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.

5 participants