DHBW-WE/MeetUp
5
0
Fork 0
Code Issues 7 Pull requests 5 Projects 1 Releases 5 Packages 1 Wiki Activity Actions

feat: set up database / orm #39

Merged
dominik merged 12 commits from feat/25-set_up_database_orm into main 2025-05-19 20:16:32 +00:00
Conversation 7 Commits 12 Files changed 11 +763 -26
micha.bok commented 2025-05-09 21:28:33 +00:00
Member
Copy link

closes #25

closes #25
micha.bok added 1 commit 2025-05-09 21:28:33 +00:00
feat: add prisma.schema content
All checks were successful
container-scan / Container Scan (pull_request) Successful in 3m52s
Details
docker-build / docker (pull_request) Successful in 3m23s
Details
ee1f4375f8
dominik changed title from feat: add prisma.schema content to feat: set up database / orm 2025-05-09 21:29:54 +00:00
dominik changed title from feat: set up database / orm to WIP: feat: set up database / orm 2025-05-09 21:30:09 +00:00
lima reviewed 2025-05-09 23:02:06 +00:00
prisma/schema.prisma Outdated
@ -16,0 +66,4 @@
model User {
id String @id @default(cuid())
username String @unique
lima commented 2025-05-09 23:02:05 +00:00
Member
Copy link

"username" durch "name" ersetzen und double checken.

https://authjs.dev/getting-started/adapters/prisma

"username" durch "name" ersetzen und double checken. https://authjs.dev/getting-started/adapters/prisma
lima marked this conversation as resolved
micha.bok added 1 commit 2025-05-11 15:21:17 +00:00
fix: change User -> username to name
All checks were successful
container-scan / Container Scan (pull_request) Successful in 8m43s
Details
docker-build / docker (pull_request) Successful in 9m29s
Details
dd35d6b956
micha.bok force-pushed feat/25-set_up_database_orm from dd35d6b956 to 97d9076415 2025-05-14 16:11:32 +00:00 Compare
dominik requested changes 2025-05-14 17:03:31 +00:00
dominik left a comment
Owner
Copy link

Siehe andere Kommentare.
Außerdem bitte noch formatieren (yarn format) sonst schlägt der build fehl.

Siehe andere Kommentare. Außerdem bitte noch formatieren (`yarn format`) sonst schlägt der build fehl.
.gitignore Outdated
@ -42,1 +42,4 @@
next-env.d.ts
# database
/prisma/dev.db
dominik commented 2025-05-14 16:59:14 +00:00
Owner
Copy link

Bitte auch den generated ordner von prisma hier excluden

src/generated/prisma
Bitte auch den generated ordner von prisma hier excluden ``` src/generated/prisma ```
micha.bok marked this conversation as resolved
prisma/schema.prisma Outdated
@ -8,2 +8,2 @@
provider = "prisma-client-js"
output = "../generated/prisma"
provider = "prisma-client-js"
output = "../generated/prisma"
dominik commented 2025-05-14 17:00:05 +00:00
Owner
Copy link

Am besten den output mit in den src Ordner packen, dann kann man das leichter importieren.

../src/generated/prisma

Am besten den output mit in den src Ordner packen, dann kann man das leichter importieren. `../src/generated/prisma`
micha.bok marked this conversation as resolved
src/auth.ts Outdated
@ -34,3 +37,3 @@
export const { handlers, signIn, signOut, auth } = NextAuth({
providers,
providers: [],
dominik commented 2025-05-14 17:01:44 +00:00
Owner
Copy link
providers,

ist das gleiche wie

providers: providers,

Die provider sind in der Datei weiter oben definiert. deshalb bitte nicht auf ein leeres Array setzen, sondern wieder auf providers.

``` providers, ``` ist das gleiche wie ``` providers: providers, ``` Die provider sind in der Datei weiter oben definiert. deshalb bitte nicht auf ein leeres Array setzen, sondern wieder auf `providers`.
micha.bok marked this conversation as resolved
src/prisma.ts Outdated
@ -0,0 +1,7 @@
import { PrismaClient } from "@prisma/client"
dominik commented 2025-05-14 17:02:32 +00:00
Owner
Copy link

Hier muss der prisma client aus dem generated Ordner importiert werden, sonst gibt es Fehlermeldungen.

@/generated/prisma
Hier muss der prisma client aus dem generated Ordner importiert werden, sonst gibt es Fehlermeldungen. ``` @/generated/prisma ```
micha.bok marked this conversation as resolved
micha.bok added 1 commit 2025-05-15 10:46:00 +00:00
fix: update prisma import paths and adjust .gitignore for generated files
Some checks failed
container-scan / Container Scan (pull_request) Failing after 11s
Details
docker-build / docker (pull_request) Failing after 1m16s
Details
ff04072045
micha.bok added 1 commit 2025-05-15 10:52:03 +00:00
style: format files
Some checks failed
container-scan / Container Scan (pull_request) Failing after 10s
Details
docker-build / docker (pull_request) Failing after 1m8s
Details
69a05e4b8b
micha.bok changed title from WIP: feat: set up database / orm to feat: set up database / orm 2025-05-15 10:52:31 +00:00
micha.bok added 1 commit 2025-05-15 10:54:11 +00:00
fix: yarn.lock
Some checks failed
container-scan / Container Scan (pull_request) Failing after 1m14s
Details
docker-build / docker (pull_request) Failing after 2m55s
Details
3509e5e5c4
micha.bok added 1 commit 2025-05-15 10:56:49 +00:00
chore: update dockerfile for prisma
Some checks failed
container-scan / Container Scan (pull_request) Failing after 47s
Details
docker-build / docker (pull_request) Failing after 3m2s
Details
c13948129b
micha.bok added 1 commit 2025-05-15 11:15:00 +00:00
fix: update eslint config to include additional ignore patterns
Some checks failed
container-scan / Container Scan (pull_request) Failing after 19s
Details
docker-build / docker (pull_request) Failing after 2m33s
Details
1a9c92275a
micha.bok force-pushed feat/25-set_up_database_orm from 1a9c92275a to 6cba56476e 2025-05-15 11:17:08 +00:00 Compare
requested review from dominik 2025-05-15 11:24:17 +00:00
dominik commented 2025-05-15 11:43:15 +00:00
Owner
Copy link

Müssen warscheinlich noch was anpassen. Der Container baut, lässt sich aber nicht starten. (Prisma Client could not locate the Query Engine for runtime "debian-openssl-3.0.x".)

Mit folgendem docker-compose.yml kann man das testen:

services:
  app:
    build:
      context: .
      dockerfile: Dockerfile
    ports:
      - '3000:3000'
    environment:
      - AUTH_SECRET=secret
      - AUTH_URL=http://localhost:3000
      - DATABASE_URL=file:/data/db.sqlite
    volumes:
      - ./data:/data

docker compose up --build
Müssen warscheinlich noch was anpassen. Der Container baut, lässt sich aber nicht starten. (`Prisma Client could not locate the Query Engine for runtime "debian-openssl-3.0.x".`) Mit folgendem docker-compose.yml kann man das testen: ```yml services: app: build: context: . dockerfile: Dockerfile ports: - '3000:3000' environment: - AUTH_SECRET=secret - AUTH_URL=http://localhost:3000 - DATABASE_URL=file:/data/db.sqlite volumes: - ./data:/data ``` ```bash docker compose up --build ```
lima added 1 commit 2025-05-15 21:09:39 +00:00
fix: update dependencies in yarn.lock
All checks were successful
container-scan / Container Scan (pull_request) Successful in 2m49s
Details
docker-build / docker (pull_request) Successful in 3m37s
Details
62633d7105
dominik commented 2025-05-19 15:27:16 +00:00
Owner
Copy link

@dominik wrote in #39 (comment):

Müssen warscheinlich noch was anpassen. Der Container baut, lässt sich aber nicht starten. (Prisma Client could not locate the Query Engine for runtime "debian-openssl-3.0.x".)

Kannst du das ganze noch mal rebasen? hab in #63 und #68 ein paar anpassungen an den docker container gemacht. Sollte dann funktionieren.

@dominik wrote in https://git.dominikstahl.dev/DHBW-WE/MeetUp/pulls/39#issuecomment-603: > Müssen warscheinlich noch was anpassen. Der Container baut, lässt sich aber nicht starten. (`Prisma Client could not locate the Query Engine for runtime "debian-openssl-3.0.x".`) Kannst du das ganze noch mal rebasen? hab in #63 und #68 ein paar anpassungen an den docker container gemacht. Sollte dann funktionieren.
micha.bok force-pushed feat/25-set_up_database_orm from 62633d7105 to db05f40c5a 2025-05-19 15:42:08 +00:00 Compare
lima approved these changes 2025-05-19 15:44:01 +00:00
Dismissed
micha.bok force-pushed feat/25-set_up_database_orm from db05f40c5a to ee2965efd6 2025-05-19 16:20:48 +00:00 Compare
lima approved these changes 2025-05-19 16:52:23 +00:00
dominik commented 2025-05-19 16:54:07 +00:00
Owner
Copy link

Musst noch eine prisma migration erstellen (DATABASE_URL="file:./dev.db" yarn prisma migrate dev sonst funktioniert das ganze in production nicht). Die Sachen im migrations Ordner müssen dann mit ins git. Dann können wir auch das mergen.

Musst noch eine prisma migration erstellen (`DATABASE_URL="file:./dev.db" yarn prisma migrate dev` sonst funktioniert das ganze in production nicht). Die Sachen im migrations Ordner müssen dann mit ins git. Dann können wir auch das mergen.
micha.bok added 2 commits 2025-05-19 19:27:49 +00:00
chore: prima init migration
All checks were successful
container-scan / Container Scan (pull_request) Successful in 4m0s
Details
docker-build / docker (pull_request) Successful in 4m42s
Details
4ccd392178
dominik force-pushed feat/25-set_up_database_orm from 4ccd392178 to e18af4f49d 2025-05-19 20:09:12 +00:00 Compare
dominik scheduled this pull request to auto merge when all checks succeed 2025-05-19 20:09:38 +00:00
dominik merged commit 72164ae8e0 into main 2025-05-19 20:16:32 +00:00
dominik deleted branch feat/25-set_up_database_orm 2025-05-19 20:16:33 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
dominik
lima
Labels
Clear labels   
Compat/Breaking

Breaking change that won't be backward compatible

  
Kind/Bug

Something is not working

  
Kind/Documentation

Documentation changes

  
Kind/Enhancement

Improve existing functionality

  
Kind/Feature

New functionality

  
Kind/Security

This is security issue

  
Kind/Testing

Issue or pull request related to testing

  
Priority
Critical
The priority is critical

  
Priority
High
The priority is high

  
Priority
Low
The priority is low

  
Priority
Medium
The priority is medium

  
Reviewed
Confirmed
Issue has been confirmed

  
Reviewed
Duplicate
This issue or pull request already exists

  
Reviewed
Invalid
Invalid issue

  
Reviewed
Won't Fix
This issue won't be fixed

  
Review needed
  
Status
Abandoned
Somebody has started to work on this but abandoned work

  
Status
Blocked
Something is blocking this issue or pull request

  
Status
Need More Info
Feedback is required to reproduce issue or to continue work

No labels
Compat/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Review needed
Status
Abandoned
Status
Blocked
Status
Need More Info
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: DHBW-WE/MeetUp#39
No description provided.
Delete branch "feat/25-set_up_database_orm"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?