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] Migration du usecase computeCertificabilitydans le BC Prescription (PIX-15342) #10693

Merged
merged 5 commits into from
Dec 4, 2024

Conversation

xav-car
Copy link
Contributor

@xav-car xav-car commented Dec 2, 2024

🍂 Problème

Il reste quelque usecase Prescription qui n'ont rien à faire dans lib

🌰 Proposition

les migrer ( à commencer par celui du calcul de la cerficabilité )

🎃 Remarques

RAS

🪵 Pour tester

Passer par les usaces impacté Accès :

  • getAccountRecoveryDetails
  • generateUsernameWithTemporaryPassword
  • updateOrganizationLearnerDependentUserPassword

Lancer le job pgboss de calcul de la certificabilité et vérfier que c'est OKAY

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

@xav-car xav-car changed the title [TECH] Migration du usecase computeCertificabilitydans le BC Prescription (PIX-XXXXX) [TECH] Migration du usecase computeCertificabilitydans le BC Prescription (PIX-15342) Dec 2, 2024
@xav-car xav-car force-pushed the migrate-prescription-usecase-part1 branch 5 times, most recently from acb86c3 to 54ed90d Compare December 2, 2024 12:43
@xav-car xav-car marked this pull request as ready for review December 2, 2024 12:54
@xav-car xav-car requested review from a team as code owners December 2, 2024 12:54
Copy link
Contributor

@alicegoarnisson alicegoarnisson left a comment

Choose a reason for hiding this comment

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

Tech OK ✅
Func Prescription OK ✅
Vu en mob 🐼 🦖 🐕 🦦

@yaf yaf added this to the Bounded context migration milestone Dec 4, 2024
Copy link
Contributor

@er-lim er-lim 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é en partie fonctionnellement.
ll reste le usecase get-account-recovery , qui est un peu plus compliqué à faire mais ce n'est pas bloquant 😄

@@ -1,11 +1,6 @@
import { knex } from '../../../db/knex-database-connection.js';
Copy link
Contributor

Choose a reason for hiding this comment

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

Remarque Côté Accès, on migre tout le contenu des repos lorsqu'on fait des migrations pour éviter de trainer avec des doublons de fichiers 😄

Est-ce qu'il ne faudrait pas faire pareil ici ? J'ai l'impression que les autres fonctions exposées ici sont aussi dans votre scope nn ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

On part plus du principe de migrer que ce qui nous intéresse.

Nos routes ayant été migré. il nous reste quelque usecase
Quand on aura migré tout nos usecases. on verra si on a des méthodes comme ça qui ne sont pas utilisé chez nous. et voir pour en faire une api interne.

@pix-service-auto-merge pix-service-auto-merge force-pushed the migrate-prescription-usecase-part1 branch from bae8f31 to 3a82dd9 Compare December 4, 2024 14:21
@pix-service-auto-merge pix-service-auto-merge merged commit 9cc5616 into dev Dec 4, 2024
8 of 9 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the migrate-prescription-usecase-part1 branch December 4, 2024 14:27
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