-
Notifications
You must be signed in to change notification settings - Fork 20
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
use TypedDocumentNode from graphql-js #68
Comments
Thank you @backbone87 for reporting this. You are right, this was introduced, but we can't change it at the moment in this package, since it will drop support for previous GraphQL versions for all libraries that are using it. I think, with time, we can gradually move it when v15/v16 becomes more popular. |
hm, is there a way to dynamically check what graphql peer version is installed? |
At the moment, no. Btw, You should be able to modify the import path of the typed |
Just chiming in that I am interested in the same. @dotansimha's comment on this PR was spot-on:
I am using To clarify: I am able to use the I'm on
These are all the latest versions. It seems like the only issue remaining is that |
TypedDocumentNode type landed in graphql-js a while ago: graphql/graphql-js#2749
since codegen (and probably some early adaptor lib authors) use this module as a source for the TypeDocumentNode type, we now end up with 2 interfaces. can this package reexport the interface from graphql-js? (and maybe deprecate this package)
The text was updated successfully, but these errors were encountered: