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 l'envoi de résultats de participations "completed" à Pôle Emploi vers le contexte Prescription (Pix-15340) #10816

Conversation

alicegoarnisson
Copy link
Contributor

@alicegoarnisson alicegoarnisson commented Dec 13, 2024

🎄 Problème

On doit déplacer le usecase send-completed-results-to-pole-emploi vers le contexte Prescription.

🎁 Proposition

On déplace les fichiers du usecase ainsi que ceux des tests et on modifie les imports.

🧦 Remarques

  • Cette MR est à merger dans shared-results-to-PE avant d'être mergée sur dev : on peut donc se dispenser de la func review

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

@alicegoarnisson alicegoarnisson self-assigned this Dec 13, 2024
@alicegoarnisson alicegoarnisson force-pushed the pix-15340/migrate-completed-results-to-PE branch from 0caf3f9 to 9af002d Compare December 13, 2024 15:35
@alicegoarnisson alicegoarnisson marked this pull request as ready for review December 13, 2024 15:42
@alicegoarnisson alicegoarnisson requested a review from a team as a code owner December 13, 2024 15:42
@alicegoarnisson alicegoarnisson changed the title Pix 15340/migrate completed results to pe [TECH]: Pix 15340 - migrer l'envoi de résultats de participations "completed" à Pôle Emploi vers le contexte Prescription Dec 13, 2024
@xav-car xav-car changed the title [TECH]: Pix 15340 - migrer l'envoi de résultats de participations "completed" à Pôle Emploi vers le contexte Prescription [TECH] Migrer l'envoi de résultats de participations "completed" à Pôle Emploi vers le contexte Prescription (Pix-15340) Dec 16, 2024
@alicegoarnisson alicegoarnisson force-pushed the pix-15340/migrate-completed-results-to-PE branch from 9af002d to 8a4819c Compare December 16, 2024 10:15
@@ -1,13 +1,13 @@
import { usecases } from '../../../../lib/domain/usecases/index.js';
import * as poleEmploiNotifier from '../../../../src/prescription/campaign-participation/infrastructure/externals/pole-emploi/pole-emploi-notifier.js';
import { usecases } from '../../../../../../lib/domain/usecases/index.js';
Copy link
Contributor

Choose a reason for hiding this comment

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

suggestion import src ?

@pix-service-auto-merge pix-service-auto-merge merged commit b98b3a9 into pix-15339/migrate-shared-results-to-PE Dec 16, 2024
10 of 11 checks passed
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.

4 participants