Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(gatsby-source-contentful): Correctly overwrite field type on Assets #36337

Merged
merged 4 commits into from
Aug 9, 2022

Conversation

LekoArts
Copy link
Contributor

@LekoArts LekoArts commented Aug 8, 2022

Description

Previously we were overriding the type of the gatsbyImageData field and subsequently the custom scalar wasn't used.

Related Issues

Fixes #36243

@LekoArts LekoArts added the topic: source-contentful Related to Gatsby's integration with Contentful label Aug 8, 2022
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 8, 2022
@LekoArts LekoArts removed the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 8, 2022
@LekoArts LekoArts changed the title fix(gatsby-source-contentful): Don't overwrite GraphQL type on Assets fix(gatsby-source-contentful): Don't overwrite field type on Assets Aug 8, 2022
wardpeet
wardpeet previously approved these changes Aug 8, 2022
@LekoArts LekoArts changed the title fix(gatsby-source-contentful): Don't overwrite field type on Assets fix(gatsby-source-contentful): Correctly overwrite field type on Assets Aug 9, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
topic: source-contentful Related to Gatsby's integration with Contentful
Projects
None yet
Development

Successfully merging this pull request may close these issues.

graphqlTypegen generating wrong Type definitions for gatsby-plugin-image + gatsby-source-contentful
2 participants