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

import utilities from the correct path #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

despairblue
Copy link

See documentation: https://graphql.org/graphql-js/utilities/

With GraphQL 15 the import path's have changed. According to the docs the things should be imported from graphql-js/utilities.

I also needed to change introspectionQuery to getIntrospectionQuery as only the latter was in the type definitions. Otherwise the tests would fail because it would not compile.

@despairblue
Copy link
Author

Not sure what the failure means. I guess I'm supposed to change the commit message, right?

@Betree
Copy link

Betree commented Dec 16, 2020

This change is required to add support for deprecated fields (see graphql/graphql-js#2855). Would be great to have it merged, but I'm not sure if this package is still maintained?

@tslater
Copy link

tslater commented Jan 2, 2021

Yeah having >1 graphql package is a problem...and this packages is getting behind. Would love to see this resolved!!!

@MarkLyck
Copy link

MarkLyck commented Jul 4, 2022

Ugh, having the same problem.

Using graphql 16.x, and gettign this error:

Error: Cannot find module 'graphql/utilities/introspectionQuery'

This has been left broken since 2020? :(

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

4 participants