-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Support overwrite of @map and @@map ERD prisma schema usage #8
base: master
Are you sure you want to change the base?
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/skn0tt/prisma-erd/32U6oBsstj83Jkf1tDxwbyEzEJSq |
Hi @keonik, thanks a ton for this PR! I checked the deploy preview, but there seems to be some error. You can run it locally by running |
@Skn0tt I think the 500 error is unrelated to the commit. The latest deployment has it commented out to see if it was the culprit. It still is getting that error. I'm playing around with updating some dependencies. Maybe something changed in the dependencies |
This reverts commit c9d3333.
Alright, the deployment works again. @keonik I'm trying to verify this actually does what it's supposed to do, and added some |
I commented it out while I was troubleshooting the 500 error. It's turned back on now. Here are separate examples. You should be able to just add test model User {
id Int @id @default(autoincrement())
hashedPassword String? @map("hashed_password")
} test model User {
id Int @id @default(autoincrement())
hashedPassword String?
@@map("users")
} |
Still doesn't appear to be working. hmm. |
Is this |
No, |
Looks like usage of the single |
I haven't used parcel before so I wasn't spending much time running this. I just copied over the function to search the data model string for
@map
and@@map
to replace in your render SVG from DML functions.