Skip to content

fix(report): tolerate open-loop config schema in ConfigCard - #208

Merged
wlawt merged 2 commits into
mainfrom
fix-ui
Jul 30, 2026
Merged

fix(report): tolerate open-loop config schema in ConfigCard#208
wlawt merged 2 commits into
mainfrom
fix-ui

Conversation

@wlawt

@wlawt wlawt commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Description

  • Producer configs may omit batch_size and emit funding_batch_size instead; skip missing fields instead of calling toLocaleString on undefined.
  • Previous deploy to test a refactor of the load tester introduced new config fields that were not accounted for in the UI causing an error on render

Testing

Producer configs may omit batch_size and emit funding_batch_size instead;
skip missing fields instead of calling toLocaleString on undefined.
@cb-heimdall

cb-heimdall commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@wlawt
wlawt enabled auto-merge (squash) July 30, 2026 17:23
@wlawt
wlawt merged commit 83589f4 into main Jul 30, 2026
15 checks passed
@wlawt
wlawt deleted the fix-ui branch July 30, 2026 18:17
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.

3 participants