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

Include local fields and types in GraphiQL #188

Merged
merged 4 commits into from
Feb 26, 2019

Conversation

justinanastos
Copy link
Contributor

@justinanastos justinanastos commented Feb 25, 2019

Resolves issue #172 and #159

Note: I've also tested this with The AC2.5 update (#166); it works as advertised 🙆

This does not allow GraphiQL to differentiate between local and remote fields; they will look the same in the Documentation Explorer.

@justinanastos justinanastos force-pushed the feat/include-local-state-in-graphiql branch from 1d87ead to 3dfb219 Compare February 26, 2019 16:36
@justinanastos justinanastos changed the base branch from master to hugh/chang/ac-local-state-changes February 26, 2019 16:37
This code is already a little confusing. Let's use some clearer variables
names.
This will augment GraphiQL's perceived schema. This will remove warnings in
the live query editor and will show client-only fields and types in the
documentation explorer.

This has no effect on field resolution logic. For Apollo Client 2.4.x and
below, querying local-only fields will still fail unless you choose to read
from the cache.
@justinanastos justinanastos force-pushed the feat/include-local-state-in-graphiql branch from 3dfb219 to 126dd59 Compare February 26, 2019 16:37
@justinanastos justinanastos changed the base branch from hugh/chang/ac-local-state-changes to master February 26, 2019 16:38
Copy link
Member

@cheapsteak cheapsteak left a comment

Choose a reason for hiding this comment

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

nice!

@justinanastos justinanastos merged commit cfb39f3 into master Feb 26, 2019
@justinanastos justinanastos deleted the feat/include-local-state-in-graphiql branch February 26, 2019 16:42
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

2 participants