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 PATCH patch /api/certification-center-invitations/{certificationCenterInvitationId} (PIX-16392) #11314

Conversation

EmmanuelleBonnemay
Copy link
Contributor

@EmmanuelleBonnemay EmmanuelleBonnemay commented Feb 3, 2025

🥞 Problème

la route PATCH patch /api/certification-center-invitations/{certificationCenterInvitationId} est encore dans /lib

🥓 Proposition

Migrer cette route dans /src/team

🧃 Remarques

Dans l'index des usecases de /lib les imports non utilisés dans /lib ne sont pas supprimés pour l'instant, mais précédés du commentaire
// Not used in lib
J'ai suivi cette pratique

😋 Pour tester

  • Se connecter sur Pix Certif avec un compte d'administrateur comme certif-pro@example.net.
  • inviter une personne dans le centre de certification (l'email n'a pas d'importance)
  • Renvoyez cette invitation (Dans le tableau des invitations, à droite de l'invitation que vous venez de faire)
  • Vérifiez que la route PATCH patch /api/certification-center-invitations/{certificationCenterInvitationId} a été appelée avec succès et qu'elle a renvoyé un objet tel que :
{
	"data": {
		"type": "certification-center-invitations",
		"id": {id de l'invitation}",
		"attributes": {
			"email": {email d'envoi de l'invitation},
			"updated-at": {date de mise à jour},
			"role": null,
			"language": "fr"
		}
	}
}

@EmmanuelleBonnemay EmmanuelleBonnemay requested a review from a team as a code owner February 3, 2025 19:50
@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 :

@EmmanuelleBonnemay EmmanuelleBonnemay force-pushed the pix-16392-patch-api-certification-center-invitations branch from 92e07d9 to d904609 Compare February 4, 2025 07:47
@EmmanuelleBonnemay EmmanuelleBonnemay self-assigned this Feb 4, 2025
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.

Testé et revu ✅

image

@yaf yaf added this to the Bounded context migration milestone Feb 5, 2025
@bpetetot bpetetot 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 5, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16392-patch-api-certification-center-invitations branch from d904609 to 047855d Compare February 5, 2025 16:54
@pix-service-auto-merge pix-service-auto-merge merged commit 3f9c8ab into dev Feb 5, 2025
8 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16392-patch-api-certification-center-invitations branch February 5, 2025 17:01
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