graphene
graphene copied to clipboard
Argument default value should be undefined
resolves #1195
This stops null
from being populated in the introspection query as mentioned in #1195. This also solves the problem of None
being passed into the kwargs of a mutate
call if no value was passed in by the end user.
I am unsure which of the two options you want me to do to fix the unit tests.
1, set default values for the failing tests to be None. 2, remove the no longer passed in values from the assertion statements.
I am unsure which of the two options you want me to do to fix the unit tests.
1, set default values for the failing tests to be None. 2, remove the no longer passed in values from the assertion statements.
thumbs up for this PR!
I would say remove the no longer passed in values from the assertion right? We're testing whether the value is absent if its not passed in
Ok I finally got around to cleaning up the changes that moving to Undefined makes to the unit tests. This should be good to go now. Fixed the fact I forgot to run pre-commit along the way.
What's the updated on this one?
+1 this seems like a critical feature
Does this PR need any more work? I'd be happy to help, otherwise I would really appreciate this to make it into the next release 😃
any update?
@Cito @jkimbo you guys probably have a lot of work on your hands, thank you so much for caring about the project! Just wanted to check in whether this change is something you consider accepting or if it needs any more work. I'd surely help making any more changes if needed.
Hey everyone,
sorry for the long wait; this fix is already merged:
https://github.com/graphql-python/graphene/commit/19ebf08339263f0446235ddffbbab0e6d3f11699
It should be included in 3.1.0
Closing this.