-
Notifications
You must be signed in to change notification settings - Fork 248
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
feat: add relation_on_update field to DMMF model #5118
base: main
Are you sure you want to change the base?
Conversation
Hey! This is my first PR, and am not that experienced with Rust, so please be patient with me 😅 This is something I wanted for one of my generator projects, so I added the missing field here, and once this is merged, I can update the types in the prisma repo to include this as well. |
For some reason the button to approve a CI run doesn't show up for me now (maybe it's because I activated the new merge experience preview feature in GitHub settings) but one thing that immediately catches my attention as something that would fail is that you need to update snapshots in tests. You need to run something like this:
|
okay, will try it, for some reason I will try it for a bit, if it doesn't work in WSL, I'll get back to this tomorrow (need to install Linux) |
4c20730
to
f794e04
Compare
I don't think WSL or bare-metal installation of Linux should make any difference. What's the exact problem you're facing, is there any specific error? |
I ran out of storage 😭. Fixing it rn! |
|
Related to: prisma/prisma#25944