-
-
Notifications
You must be signed in to change notification settings - Fork 562
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
Default values not work for notNull fields #274
Comments
This is an issue in spec and reference implementation. See graphql/graphql-js#1274 We will likely use the same solution eventually. |
Since the PR on graphql-js has been merged, will the behaviour be ported to this library? |
Yes, but this will be likely a part of 0.13 release |
Thank you for the quick reply. |
Unfortunately no, we've just released 0.12. But chances are high that it will take much less time than 0.12 %) |
Hehe, okay. Thanks! |
@vladar So will this be in 0.13 then? I noticed in another thread that you want to release it soon. |
I did plan to include it in 0.13 but have little time now. And we already delayed the release for a while. Also in the reference implementation, it is a part of the 14.0.0 release. We could include it in our 0.13 if someone ported these changes in the next couple of days. I'll try to find time but can't promise it. |
Field with type 'notNull' ignores default value and throws error:
"Argument "paging" has invalid value {pageSize: 2}. In field "page": Expected "Int!", found null."
With nullable types default value works as expected but then client can pass null value into page field...
The text was updated successfully, but these errors were encountered: