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 DELETE /api/admin/certification-center-memberships/{id} (PIX-16387) #11315

Conversation

P-Jeremy
Copy link
Contributor

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

🥞 Problème

la route DELETE patch /api/admin/certification-center-memberships/{id} est encore dans /lib

🥓 Proposition

Migrer cette route dans /src/team

🧃 Remarques

😋 Pour tester

Depuis pix-admin

  • Se rendre sur la page de détail d'un centre de certification
  • Dans l'onglet "Equipe", tenter de désactiver un membre
  • Constater le message de succès

@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 4, 2025
@bpetetot bpetetot force-pushed the pix-16387-migrate-admin-certification-center-memberships-delete-route branch from 499c8f7 to 1dc7ba0 Compare February 4, 2025 14:21
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

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

Revu et testé ✅
image

@yaf yaf added this to the Bounded context migration milestone Feb 5, 2025
@P-Jeremy P-Jeremy force-pushed the pix-16387-migrate-admin-certification-center-memberships-delete-route branch 2 times, most recently from cc875b0 to cae4758 Compare February 5, 2025 08:57
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 force-pushed the pix-16387-migrate-admin-certification-center-memberships-delete-route branch from cae4758 to b0324b5 Compare February 5, 2025 08:59
@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 5, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16387-migrate-admin-certification-center-memberships-delete-route branch from b0324b5 to a193a16 Compare February 5, 2025 09:12
@pix-service-auto-merge pix-service-auto-merge merged commit fe2a07a into dev Feb 5, 2025
8 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16387-migrate-admin-certification-center-memberships-delete-route branch February 5, 2025 09:18
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