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

Fix non-local schema not loading #91

Merged
merged 3 commits into from
Sep 17, 2024
Merged

Conversation

mizdra
Copy link
Owner

@mizdra mizdra commented Sep 16, 2024

close: #87

The getTypeInfos function, which extracts schema type information, depends on the GraphQLNamedType#astNode property. However, the schema fetched by introspection does not have that property. Therefore, the code generation was broken.

This Pull Request changes the getTypeInfos function to not depend on the GraphQLNamedType#astNode property.

@mizdra mizdra added the Type: Fix Fix bug. label Sep 16, 2024
@mizdra mizdra marked this pull request as draft September 16, 2024 02:41
@mizdra mizdra force-pushed the fix-compatibility-for-remote-schema branch from 49651de to 120b7cf Compare September 17, 2024 12:47
@mizdra mizdra marked this pull request as ready for review September 17, 2024 13:34
@mizdra mizdra merged commit c99215f into main Sep 17, 2024
6 checks passed
@mizdra mizdra deleted the fix-compatibility-for-remote-schema branch September 17, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Fix Fix bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to fetch factory when using a URL pointing to a GraphQL endpoint
1 participant