Skip to content

fix(locale-field-populator): remove 50-entry cap [ES-183]#10970

Merged
Adrian Meyer (primeinteger) merged 3 commits into
masterfrom
ES-183/entry-limit
May 11, 2026
Merged

fix(locale-field-populator): remove 50-entry cap [ES-183]#10970
Adrian Meyer (primeinteger) merged 3 commits into
masterfrom
ES-183/entry-limit

Conversation

@primeinteger
Copy link
Copy Markdown
Contributor

@primeinteger Adrian Meyer (primeinteger) commented May 11, 2026

Purpose

Remove the hard-coded 50-entry cap that prevents customers with large reference graphs from using the Locale Field Populator, and add progressive rendering to keep the preview UI performant at scale.

Approach

  • Removed MAX_TOTAL_ENTRIES entirely -- the existing visited-set (cycle prevention) and configurable maxDepth already guarantee termination, making the cap redundant
  • Added chunked getMany calls (batches of 100) and concurrency-limited updates (5 parallel) to stay within CMA rate limits at higher entry counts
  • Added a summary banner showing total referenced entries and localizable field counts so users always see the full scope
  • Render first 50 entries with incremental "Show more" pagination to avoid mounting hundreds of accordion sections at once

Screenshots

Showing the summary banner: "61 referenced entries with 64 localizable fields"

Screenshot 2026-05-11 at 10 28 23 AM

Showing the load more button: "Load 11 more entries"

Screenshot 2026-05-11 at 10 28 43 AM

Copy link
Copy Markdown
Contributor

@mgoudy91 Mitch Goudy (mgoudy91) left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@primeinteger Adrian Meyer (primeinteger) merged commit 94b9bd9 into master May 11, 2026
14 checks passed
@primeinteger Adrian Meyer (primeinteger) deleted the ES-183/entry-limit branch May 11, 2026 19:31
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.

2 participants