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

typescript-client: Include __typename in fragments and metadata fields #1380

Merged
merged 3 commits into from
Mar 1, 2019

Conversation

kamilkisiela
Copy link
Collaborator

@kamilkisiela kamilkisiela commented Feb 28, 2019

Fixesss (@dotansimha related) #643 (comment) and #643 (comment)

@kamilkisiela
Copy link
Collaborator Author

I don't know if that works within fragments but I think it's enough for now

@dotansimha
Copy link
Owner

Awesome! ❤️ @kamilkisiela
Can you please implement the same for the new plugins? (#1353)

@dotansimha dotansimha merged commit 03e18d3 into master Mar 1, 2019
@kamilkisiela kamilkisiela deleted the feat/__typename branch March 1, 2019 09:53
@kamilkisiela
Copy link
Collaborator Author

kamilkisiela commented Mar 1, 2019

@dotansimha ❤️ of course!

@awdyson
Copy link

awdyson commented Mar 8, 2019

@kamilkisiela thanks for the work, this helped me quite a bit

Are you planning on adding __typename to the typescript-server plugin as well?
Consider this a vote for if you're on the fence 😛

@awdyson
Copy link

awdyson commented Mar 8, 2019

Playing with the canary build, I guess my true ask would be to (optionally?) add __typename to the typescript plugin.

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.

3 participants