-
Notifications
You must be signed in to change notification settings - Fork 272
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
a __typename field at the Query level should receive a matching response #1922
Labels
Comments
That's likely yes |
fixed by #2253 |
It looks like I still see the issue with query CanDeferFragmentsOnTheTopLevelQueryFieldQuery {
__typename
...FragmentOnQuery @defer
}
fragment FragmentOnQuery on Query {
computers {
id
}
} Repro steps: git clone git@github.com:apollographql/apollo-kotlin.git
cd apollo-kotlin
git checkout ed8a6ec9d95697545b6211decb518aa4a8fd6fee
# Install and run the subgraph
(cd tests/defer/router/subgraphs/computers && npm install && APOLLO_PORT=4001 npm start)&
# Run the router
path/to/router --supergraph tests/defer/router/simple-supergraph.graphqls &
# Query
curl --request POST --header 'content-type: application/json' --header 'accept: multipart/mixed; deferSpec=20220824, application/json' --url 'http://127.0.0.1:4000/' --data '{"query":"query CanDeferFragmentsOnTheTopLevelQueryFieldQuery {\n __typename\n ...FragmentOnQuery @defer\n}\n\nfragment FragmentOnQuery on Query {\n computers {\n id\n }\n}"}' Actual:
Expected:
|
I'm able to reproduce it, I will investigate |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
in #1800 (comment), a deferred query is done where the
__typename
field is not added in the right response
The text was updated successfully, but these errors were encountered: