Feat: Implemented Message Module Feature - Closing Issue#243#251
Feat: Implemented Message Module Feature - Closing Issue#243#251derianrddev merged 4 commits intoKaizenode:mainfrom
Conversation
derianrddev
left a comment
There was a problem hiding this comment.
Hello @Shukazuby
Just a few small adjustments needed regarding the Prisma model and unnecessary dependencies.
Once those are addressed, everything should be ready to merge!
| } | ||
|
|
||
|
|
||
| model Message { |
There was a problem hiding this comment.
To keep consistency with other Prisma models, please update the Message model to use camelCase field names with @map for database columns.
Example:
model Message {
id String @id @default(uuid())
orderId String @map("order_id")
senderAddress String @map("sender_address")
content String
createdAt DateTime @default(now()) @map("created_at")
updatedAt DateTime @updatedAt @map("updated_at")
order Order @relation(fields: [orderId], references: [id])
sender User @relation("UserMessages", fields: [senderAddress], references: [walletAddress])
@@map("messages")
}Update the service and tests to match the new field names.
There was a problem hiding this comment.
It looks like none of the newly added dependencies in apps/backend/package.json are necessary for this PR:
@types/react@types/react-dom@types/lodash@types/trusted-types@types/web
Please remove them to keep the project clean.
If any of them are actually required for the work done, feel free to leave those in.
|
Hi @derianrddev. Thank you, I have effected the changes |
derianrddev
left a comment
There was a problem hiding this comment.
Great job! Thank you very much!
📝 Pull Request Title
🛠️ Issue
📖 Description
✅ Changes made
🖼️ Media (screenshots/videos)
📜 Additional Notes