-
-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat: required logged in to add question #447
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
📦 Next.js Bundle AnalysisThis analysis was generated by the next.js bundle analysis action 🤖 This PR introduced no changes to the javascript bundle 🙌 |
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.
Device | URL |
---|---|
mobile | https://devfaq-dha7nfczq-typeofweb.vercel.app |
Not what you expected? Are your scores flaky? GitHub runners could be the cause.
Try running on Foo instead
apps/api/prisma/schema.prisma
Outdated
@@ -14,11 +14,13 @@ model Question { | |||
categoryId String @map("_categoryId") | |||
levelId String @map("_levelId") | |||
statusId String @default("pending") @map("_statusId") | |||
userId Int @map("_userId") |
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.
userId
niewiele mówi – powinniśmy zmienić nazwę tego pola na createdById
apps/api/prisma/schema.prisma
Outdated
createdAt DateTime @default(now()) @db.Timestamptz(6) | ||
updatedAt DateTime @updatedAt() @db.Timestamptz(6) | ||
QuestionCategory QuestionCategory @relation(fields: [categoryId], references: [id], onDelete: Cascade) | ||
QuestionLevel QuestionLevel @relation(fields: [levelId], references: [id], onDelete: Cascade) | ||
QuestionStatus QuestionStatus @relation(fields: [statusId], references: [id], onDelete: Cascade) | ||
User User @relation(fields: [userId], references: [id]) |
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.
Tak samo tutaj, chyba CreatedBy
byłoby lepsze niż User
@@ -5,8 +5,9 @@ CREATE TABLE "Question" ( | |||
"acceptedAt" TIMESTAMPTZ(6), |
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.
Nie można modyfikować już istniejących migracji.
…quired-logged-in-to-add-question
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.
Device | URL |
---|---|
mobile | https://devfaq-qekx774eu-typeofweb.vercel.app |
Not what you expected? Are your scores flaky? GitHub runners could be the cause.
Try running on Foo instead
Fixes #445