Draft: move to Prisma ORM#125
Draft
HunteRoi wants to merge 11 commits into
Draft
Conversation
HunteRoi
force-pushed
the
111-change-orm-to-prisma
branch
from
d01120d to
bd6d2b3
Compare
added 9 commits
May 28, 2026 00:09
…orkflows - Introduced architecture.md detailing system context, high-level diagram, tenant access flow, and cross-entrypoint consistency. - Created bpmn-lifecycle.md outlining the onboarding and eligibility process with a BPMN-style flow diagram. - Added discord-sources.md to reference authoritative Discord API and Discord.js documentation. - Established github-sources.md to track project-level GitHub issues and repository references. - Developed issue-93-specification.md as an umbrella epic for bot refactor and web app delivery, including MVP definition and implementation phases. - Defined mvp-scope.md to clarify in-scope and out-of-scope features for the MVP. - Outlined non-mvp-roadmap.md for planned features post-MVP and suggested delivery order. - Documented state-machine.md to illustrate the membership lifecycle and critical guard rules. - Created traceability-matrix.md to map rules to specifications, GitHub issues, and implementation targets.
… and command deployment chore: modify docker-compose command to use entrypoint script refactor: simplify index.ts by removing config read and initializing DatadropClient with intents only fix: remove env-gen script from package.json and update dotenvx dependency version feat: enhance prisma.config.ts to support external tables for cron jobs delete: remove old migration SQL file and create new migration for users and guild configurations fix: update guild member events to fetch configuration dynamically fix: update command handler to fetch configuration for authorization checks feat: implement deploy command for synchronizing slash commands with Discord chore: update PrismaDatabaseService to use PostgreSQL adapter and improve configuration handling
AugustinMauroy
left a comment
AugustinMauroy
left a comment
There was a problem hiding this comment.
Autre truc que tu peux utiliser pour simplifier ton workfow de deployment the le node --run à la place de yarn
| strategy: | ||
| matrix: | ||
| node-version: [20.x] | ||
| node-version: [25.x] |
There was a problem hiding this comment.
pk une matrix de une seul version utilise la LTS
|
|
||
| const database = new PrismaClient(); | ||
|
|
||
| async function main() { |
| "type": "module", | ||
| "main": "./build/index.js", | ||
| "prisma": { | ||
| "seed": "tsx prisma/seed.ts" |
There was a problem hiding this comment.
Suggested change
| "seed": "tsx prisma/seed.ts" | |
| "seed": "node prisma/seed.ts" |
sur les dernières version de node tu peux run du typescript sans problème
| @@ -0,0 +1,20 @@ | |||
| // This file was generated by Prisma, and assumes you have installed the following: | |||
| // npm install --save-dev prisma dotenv | |||
| import "dotenv/config"; | |||
There was a problem hiding this comment.
Suggested change
| import "dotenv/config"; |
| // npm install --save-dev prisma dotenv | ||
| import "dotenv/config"; | ||
| import { defineConfig } from "prisma/config"; | ||
|
|
There was a problem hiding this comment.
https://nodejs.org/docs/latest/api/process.html#processloadenvfilepath
Suggested change
| loadEnvFile(); |
| "noImplicitAny": false, | ||
| "rootDir": ".", | ||
| "outDir": "build", | ||
| "resolvePackageJsonImports": true |
There was a problem hiding this comment.
Suggested change
| "resolvePackageJsonImports": true | |
| "resolvePackageJsonImports": true, | |
| "erasableSyntaxOnly": true |
| "homepage": "https://github.com/section-IG/DataDrop#readme", | ||
| "dependencies": { | ||
| "@dotenvx/dotenvx": "^1.51.0", | ||
| "@dotenvx/dotenvx": "^1.71.2", |
There was a problem hiding this comment.
Suggested change
| "@dotenvx/dotenvx": "^1.71.2", |
| "@types/node": "^25.9.2", | ||
| "@types/nodemailer": "^8.0.0", | ||
| "prisma": "^7.8.0", | ||
| "tsx": "^4.22.4", |
There was a problem hiding this comment.
Suggested change
| "tsx": "^4.22.4", |
| "rootDir": ".", | ||
| "outDir": "build", | ||
| "resolvePackageJsonImports": true | ||
| } |
There was a problem hiding this comment.
ts vas fails, tu dois utiliser "types": []
|
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
No description provided.