Skip to content

ORM migration from Drizzle to Prisma#102

Open
Eylexander wants to merge 4 commits into
devfrom
feat/prisma-mig
Open

ORM migration from Drizzle to Prisma#102
Eylexander wants to merge 4 commits into
devfrom
feat/prisma-mig

Conversation

@Eylexander

Copy link
Copy Markdown

J'ai globalement suivi la doc officielle de la migration + j'ai suivi ce qui se faisait côté API du Site Etu.

En soit c'est plus ajout de Prisma qu'une transition réelle, j'ai quand même laissé l'ancien code de Drizzle au besoin, mais ça se supprime vite. En soit le travail était surtout dans le changement des services du backend.

@Eylexander Eylexander requested a review from tuturd July 6, 2026 13:16
@tuturd

tuturd commented Jul 7, 2026

Copy link
Copy Markdown
Member

Ca me semble bien propre !

J'ai ajouté les configs Prettier qu'on ajoute sur toutes les branches (elles sont maintenant en dev), il faudra aller dans les fichiers modifiés par la PR pour que les styles automatiques s'appliquent via l'extension Prettier.

Sur le fonctionnement, le découpage en fichiers différents des schémas me semble cohérent.
Il faut maintenant se balader dans toutes les fonctions du site pour vérifier que les requêtes sont les bonnes. Les types devraient nous signaler les problèmes en théorie, mais on ne sait jamais.
Je suis partisan du fait de laisser la config drizzle en place pour l'instant.

Plus tard, il faudra réfléchir de manière plus approfondi sur notre db, certains designs, critères d'unicité ou caractères optionnels sont illogiques.

Selon moi il faut maintenant faire du test sur les fonctionnalités, et attendre que les autres groses PRs soient migrées en dev, puis rebase et remodifier les nouvelles requêtes drizzle ajoutées entre temps.

Good job !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants