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 /api/admin/organizations/{id}/archive (PIX-16755) #11526

Merged

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Feb 26, 2025

🥞 Problème

La route POST /api/admin/organizations/{id}/archive n'est pas encore migrée dans le contexte organization-entities

🥓 Proposition

Déplacer la route POST /api/admin/organizations/{id}/archive dans le contexte `organization-entities

🧃 Remarques

Le test unitaire du use case a été modifié en test d'intégration.

😋 Pour tester

  1. Se connecter à Pix Admin en tant que superadmin
  2. Aller sur une page d'une organisation
  3. Cliquer sur le bouton "Archiver l'organisation"
  4. Vérifier que:
  • La notification Cette organisation a bien été archivée. s'affiche
  • La phrase suivante apparait dans les détails de l'orga Archivée le 26/02/2025 par Admin Admin.

@bpetetot bpetetot self-assigned this Feb 26, 2025
@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 :

@bpetetot bpetetot marked this pull request as ready for review February 26, 2025 16:36
@bpetetot bpetetot requested a review from a team as a code owner February 26, 2025 16:36
@bpetetot bpetetot force-pushed the pix-16755-migrate-archive-organisation-route branch from 1c7911a to a3c4454 Compare February 26, 2025 16:50
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 this to the Bounded context migration milestone Feb 28, 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 28, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16755-migrate-archive-organisation-route branch from a3c4454 to 5cc5586 Compare February 28, 2025 08:50
@pix-service-auto-merge pix-service-auto-merge merged commit 1861e01 into dev Feb 28, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16755-migrate-archive-organisation-route branch February 28, 2025 08:56
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