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 Admin qui liste les adhésions aux centres de certification d'un utilisateur (PIX-16452) #11386

Conversation

Libouk
Copy link
Member

@Libouk Libouk commented Feb 10, 2025

🥞 Problème

Cette route permet de lister les centres de certification dont l'utilisateur renseigné est un membre. Dans le cadre de la migration DDD, on souhaite migrer cette route avec son controller et usecase vers le domaine Team.

🥓 Proposition

Migrer la route, le controller et le usecase de cette route et leurs tests.

😋 Pour tester

  • Se rendre sur Pix Admin
  • Aller sur la page de détail d'un utilisateur membre d'un ou plusieurs centres de Certification (L'utilisateur 7104 par exemple)
  • Cliquer sur l'onglet "Pix Certif"
  • Constater que le ou les centres de Certification dont l'utilisateur est membre est ou sont listés
  • La version rapide

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

@Libouk Libouk force-pushed the pix-16452-move-admin-certification-center-memberships-route branch 3 times, most recently from 5098aa5 to 64911c2 Compare February 11, 2025 09:13
@Libouk Libouk added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally and removed Development in progress labels Feb 11, 2025
@Libouk Libouk force-pushed the pix-16452-move-admin-certification-center-memberships-route branch from 64911c2 to 6b5bb09 Compare February 11, 2025 09:59
@lego-technix lego-technix self-requested a review February 11, 2025 10:25
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 🦊

@Libouk Libouk force-pushed the pix-16452-move-admin-certification-center-memberships-route branch from 6b5bb09 to f08c41f Compare February 11, 2025 13:04
@Libouk Libouk force-pushed the pix-16452-move-admin-certification-center-memberships-route branch from f08c41f to dc2a099 Compare February 12, 2025 16:33
@Libouk Libouk added Tech Review OK 🚀 Ready to Merge Func Review OK PO validated functionally the PR and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Feb 13, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16452-move-admin-certification-center-memberships-route branch from dc2a099 to 3f2098e Compare February 13, 2025 09:40
@Libouk Libouk force-pushed the pix-16452-move-admin-certification-center-memberships-route branch from 3f2098e to dc2a099 Compare February 13, 2025 10:42
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16452-move-admin-certification-center-memberships-route branch from dc2a099 to 88e575e Compare February 13, 2025 10:43
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16452-move-admin-certification-center-memberships-route branch from 88e575e to 238ba92 Compare February 14, 2025 10:03
@pix-service-auto-merge pix-service-auto-merge merged commit a64d475 into dev Feb 14, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16452-move-admin-certification-center-memberships-route branch February 14, 2025 10:08
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.

7 participants