-
Notifications
You must be signed in to change notification settings - Fork 57
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] Retourner tous les fichiers XML en READY_TO_SEND dans le mail CPF à destination du métier (PIX-9102). #7499
Merged
pix-service-auto-merge
merged 7 commits into
dev
from
pix-9102-send-email-after-cpf-files-generation
Nov 27, 2023
Merged
[TECH] Retourner tous les fichiers XML en READY_TO_SEND dans le mail CPF à destination du métier (PIX-9102). #7499
pix-service-auto-merge
merged 7 commits into
dev
from
pix-9102-send-email-after-cpf-files-generation
Nov 27, 2023
+83
−187
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Une fois les applications déployées, elles seront accessibles via les liens suivants : Les variables d'environnement seront accessibles via les liens suivants : |
70b41fa
to
36f3b9e
Compare
e89677d
to
ea19ed0
Compare
ea19ed0
to
a008ae4
Compare
a008ae4
to
9d01334
Compare
Proposition (pas de moi): Mettre à jour le |
aedf8fd
to
4a7ff5f
Compare
Steph0
reviewed
Nov 24, 2023
api/src/certification/session/domain/usecases/get-cpf-presigned-urls.js
Outdated
Show resolved
Hide resolved
api/tests/certification/session/integration/domain/usecases/get-cpf-presigned-urls_test.js
Show resolved
Hide resolved
b4af8c8
to
2a04df4
Compare
aceol
approved these changes
Nov 27, 2023
Steph0
approved these changes
Nov 27, 2023
2a04df4
to
8c0ac37
Compare
Co-authored-by: Andreia Pena <andreia.pena@pix.fr> Co-authored-by: Jérémy Pluquet <jemyplu@gmail.com> Co-authored-by: Antoine Ceol <antoine.ceol@gmail.com> Co-authored-by: Steph0 <Steph0@users.noreply.github.com> Co-authored-by: Jérémie Picard <jeremie.picard@pix.fr>
Co-authored-by: Andreia Pena <andreia.pena@pix.fr> Co-authored-by: Jérémy Pluquet <jemyplu@gmail.com> Co-authored-by: Antoine Ceol <antoine.ceol@gmail.com>
60a87b2
to
b9a0478
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🦄 Problème
Actuellement dans notre process CPF, l'envoi du mail contenant le lien de téléchargement des fichiers XML pose problème. En effet, on ne retourne dans ce lien que les fichiers générés à la date du cron du planner.
Mais si la génération des fichiers XML n’est pas terminé lors du demarrage de l'envoi de l'email alors ce dernier sera incomplet (n'aura pas l'ensemble des fichiers générés).
🤖 Proposition
Ne plus dépendre d'une date ( géré par des systèmes externes, peu maîtrisable ) mais du status
READY_TO_SEND
enregistré en BDD (importStatus
de la tablecertification-course-cpf-infos
).On va ainsi envoyer tous les fichiers prêts à être transmis au CPF en lien du mail envoyé au métier.
A noter que les fichiers non traités depuis le dernier email seront ajouté dans le nouvel email avec une lien présigné mis à jour.
🌈 Remarques
Avec cette proposition, la problématique de fréquence entre les deux CRON n'en est plus une. Même si l'email est envoyé avant la génération des fichiers, on pourra facilement enclencher l'envoi d'un autre mail plus tard qui se chargera de retourner les fichiers manquants.
La fréquence de ces CRON sera juste repensé.
Plutôt que de chercher à lancer le CRON de l'email quelques heures après la génération du mail, on peut decoreller ceux ci et eviter par ailleurs des envois la nuit.
On propose de mettre le CRON d'envoi d'email toute les semaines et de mettre l'expiration de liens sur 1 semaine:
CPF_EXPORTS_STORAGE_PRE_SIGNED_EXPIRES_IN=604800
(1 semaine)CPF_SEND_EMAIL_JOB_CRON=0 0 1 1 *=00 08 * * 1
(tout les lundi à 8H)💯 Pour tester
Lancer la creation d'un fichier (s'assurer du bucket dans les varenv):
Une fois terminé, redemarrer avec le mailing avec un CRON qui demarre juste apres (ici 12H28)
```update "certification-courses-cpf-infos" set "importStatus"='SUCCESS' LIMIT 1;````
Une fois terminé, redemarrer avec le mailing avec un CRON qui demarre juste apres (ici 12H35)
S'assurer que le mail recu ne contient plus le fichier en succes mais bien les autres