-
Notifications
You must be signed in to change notification settings - Fork 51
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] sépare le usecase de remplacement des learner (Pix-12529) #8931
[TECH] sépare le usecase de remplacement des learner (Pix-12529) #8931
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
i18n: request.i18n, | ||
}); | ||
await usecases.validateSupCsvFile({ | ||
organizationId, | ||
i18n: request.i18n, | ||
}); | ||
warnings = await usecases.importSupOrganizationLearners({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion au début j'ai fait euuuuuuuu pourquoi. avant de me dire mais oui c'est stocker dans organization-import
maintenant.
Note pour le suivant qui se posera la questiuon
@@ -81,9 +80,7 @@ const replaceSupOrganizationLearners = async function ( | |||
} | |||
} | |||
|
|||
return h | |||
.response(dependencies.supOrganizationLearnerWarningSerializer.serialize({ id: organizationId, warnings })) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
supprimer l'import / dependencies maintenant :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tech ok mob 🐼 🐕 🦉
e4ce249
to
c1057f2
Compare
🦄 Problème
On souhaite passer l'import du sup en asynchrone, pour cela, on doit découper le usecase en 3 partie (envoi / validation / import) afin de pouvoir ensuite en faire des taches asynchrone pour pgboss
🤖 Proposition
On split le gros usecase en réutilisant les usecases déjà utilisés pour l'import sup
🌈 Remarques
On profite de la PR pour mieux gérer les warning qu'on avait un peu passé rapidement
💯 Pour tester
importer