Skip to content
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

Solving issue #31 #35

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Solving issue #31 #35

wants to merge 4 commits into from

Conversation

lorado
Copy link
Contributor

@lorado lorado commented Dec 13, 2018

Hi.

As I already described in my issue #31, almost all helper tools for working with GraphQL claiming at __typename on InputObjects, so I just renamed it to _morphType. With this change it is directly clear, what this field means, and it also don't use any naming reserved by GraphQL anymore.

Though I have to mention - this PR has brake-changes to some users, if they already used __typename on input.

This reverts commit 82432e6
Sorry, my dependencies were not updated, so I got other output... My changes were wrong, so I reverted them
@lorado
Copy link
Contributor Author

lorado commented Jan 10, 2019

@cmizzi Happy new year!

Can you please merge this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant