Skip to content

fix: fix processAssetForLocale() bug, where a param was incorrectly n… [DX-1161]#1638

Merged
ethan ozelius (ethan-ozelius-contentful) merged 1 commit into
mainfrom
fix/incorrect-param-process-for-locale
May 27, 2026
Merged

fix: fix processAssetForLocale() bug, where a param was incorrectly n… [DX-1161]#1638
ethan ozelius (ethan-ozelius-contentful) merged 1 commit into
mainfrom
fix/incorrect-param-process-for-locale

Conversation

@ethan-ozelius-contentful
Copy link
Copy Markdown
Contributor

@ethan-ozelius-contentful ethan ozelius (ethan-ozelius-contentful) commented May 26, 2026 •
edited
Loading

As titled, see this github issue for more context: #1632

Bug 1: retryLimit is silently ignored because contentful-import passes
processingCheckRetry (singular) to processForLocale, but contentful-management
expects processingCheckRetries (plural). The unknown key is discarded and the SDK falls
back to its default of 10 retries regardless of the configured value.

PR Checklist

  • I have read the CONTRIBUTING.md file
  • All commits follow conventional commits
  • Documentation is updated (if necessary)
  • PR doesn't contain any sensitive information
  • There are no breaking changes

@ethan-ozelius-contentful ethan ozelius (ethan-ozelius-contentful) changed the title fix: fix processAssetForLocale() bug, where a param was incorrectly n… fix: fix processAssetForLocale() bug, where a param was incorrectly n… [DX-1161] May 26, 2026
@ethan-ozelius-contentful ethan ozelius (ethan-ozelius-contentful) force-pushed the fix/incorrect-param-process-for-locale branch 2 times, most recently from 9234006 to a636745 Compare May 26, 2026 22:21
Open
@ethan-ozelius-contentful ethan ozelius (ethan-ozelius-contentful) force-pushed the fix/incorrect-param-process-for-locale branch from a636745 to fdf145a Compare May 27, 2026 15:27
@ethan-ozelius-contentful ethan ozelius (ethan-ozelius-contentful) merged commit f262928 into main May 27, 2026
12 checks passed
@ethan-ozelius-contentful ethan ozelius (ethan-ozelius-contentful) deleted the fix/incorrect-param-process-for-locale branch May 27, 2026 21:01
@franko-c dar1en (franko-c) mentioned this pull request May 28, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tylerpina Tyler Pina (tylerpina) tylerpina approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ethan-ozelius-contentful @tylerpina