-
Notifications
You must be signed in to change notification settings - Fork 212
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 push
scalar list operation for update
#1753
Conversation
a1a8a5a
to
aeaceb2
Compare
// .map(|f| self.model().fields().find_from_scalar(f.as_str()).unwrap()) | ||
// .collect() | ||
// } | ||
// } else { |
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.
Seems like a leftover from the mongo PR, can be removed if you have time.
@@ -168,7 +196,62 @@ class ScalarListsSpec extends FlatSpec with Matchers with ApiSpecBase { | |||
|}""", | |||
project = project, | |||
errorCode = 2009, | |||
errorContains = """`Mutation.createScalarModel.data.ScalarModelCreateInput.strings.ScalarModelCreatestringsInput.set`: A value is required but not set.""" |
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.
This is unexpected, how did the default type flip for Creates?
model_name: String, | ||
input_object_name: T, | ||
f: ScalarFieldRef, | ||
include_set_only: bool, |
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.
If you want to distinguish between update and create operation sets, then I suggest naming it accordingly.
is_create
or similar, for example.
Closes prisma/prisma#5078