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: contentful delta updates #38468

Merged
merged 3 commits into from Aug 22, 2023
Merged

fix: contentful delta updates #38468

merged 3 commits into from Aug 22, 2023

Conversation

axe312ger
Copy link
Collaborator

We forgot to disable link resolving when delta updates are done. Page limit was also not respected.

This ensures both parameters are now passed to each sync call, probably fixing a lot of issues people have with Gatsby Cloud.

This might fix #38203

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Aug 22, 2023
@axe312ger axe312ger added type: bug An issue or pull request relating to a bug in Gatsby topic: source-contentful Related to Gatsby's integration with Contentful and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Aug 22, 2023
@axe312ger axe312ger added this to In progress in Contentful source plugin improvements via automation Aug 22, 2023
@axe312ger axe312ger self-assigned this Aug 22, 2023
@axe312ger axe312ger requested a review from pieh August 22, 2023 11:34
pieh
pieh previously approved these changes Aug 22, 2023
@pieh pieh added the bot: merge on green Gatsbot will merge these PRs automatically when all tests passes label Aug 22, 2023
@axe312ger
Copy link
Collaborator Author

This was overlooked in #37981. Sorry for not catching it in review 😢

@gatsbybot gatsbybot merged commit 51120f4 into master Aug 22, 2023
32 checks passed
Contentful source plugin improvements automation moved this from In progress to Done Aug 22, 2023
@gatsbybot gatsbybot deleted the fix/contentful-delta-updates branch August 22, 2023 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: merge on green Gatsbot will merge these PRs automatically when all tests passes topic: source-contentful Related to Gatsby's integration with Contentful type: bug An issue or pull request relating to a bug in Gatsby
3 participants