Skip to content
This repository was archived by the owner on Apr 13, 2023. It is now read-only.

Conversation

ZhengYuTay
Copy link
Contributor

@ZhengYuTay ZhengYuTay commented May 22, 2019

Fixes #1569.

The problem is when we did not implicitly set a nullable variable value to a mutation query. It will be set to null which is not intended.

@apollo-cla
Copy link

@ZhengYuTay: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Meteor Contributor Agreement here: https://contribute.meteor.com/

@axax
Copy link

axax commented May 27, 2019

@ZhengYuTay Thanks for the pull request. Fix is also for issue #2890. Hope we can merge this soon.

Copy link
Member

@hwillson hwillson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks very much @ZhengYuTay!

@hwillson hwillson merged commit df803e9 into apollographql:master May 28, 2019
@johansteffner
Copy link

@hwillson do you have any eta on when this fix will be released?

@yashrajpchavda
Copy link

yashrajpchavda commented Dec 3, 2019

@hwillson same question as @johansteffner asked, any idea when this be released? I have [email protected] and I still see this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Optional input is set to null

6 participants