-
Notifications
You must be signed in to change notification settings - Fork 59
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
Scalar lists are not queried if the info object is omitted #56
Comments
Change required here: https://github.com/graphql-binding/graphql-binding/blob/master/src/info.ts#L36-L47. Use the utility functions from https://github.com/graphql/graphql-js/blob/master/src/type/definition.js to get the correct types ( Also, this is not a bug, but an enhancement, as a scalar list field is not a plain scalar field. |
Thank you for reporting. In the last few months, since the transition of many libraries under The Guild's leadership, We've reviewed and released many improvements and versions to graphql-cli, graphql-config and graphql-import. We've reviewed What we've found is that the new GraphQL Mesh library is covering not only all the current capabilities of GraphQL Binding, but also the future ideas that were introduced in the original GraphQL Binding blog post and haven't come to life yet. And the best thing - GraphQL Mesh gives you all those capabilities, even if your source is not a GraphQL service at all! Just like GraphQL Binding, you get a fully typed SDK (thanks to the protocols SDKs and the GraphQL Code Generator), but from any source, and that SDK can run anywhere, as a connector or as a full blown gateway. If you think that we've missed anything from GraphQL Binding that is not supported in a better way in GraphQL Mesh, please let us know! |
Omitting the
info
object from a query is supposed to fetch all scalar fields. This does not seem to be the case for scalar list fields.Consider this Prisma data model:
If I run this resolver:
The result contains
id
andtitle
, but notpage
.The text was updated successfully, but these errors were encountered: