scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-10-06 02:48:13 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 103

fix: treat empty router settings inputs as null instead of preserving old values

Browse source 
When a user clears a numeric field in the RouterSettingsAccordion, the empty
input should result in null, not fall through to the previous value from state.
This commit is contained in:
Yuneng Jiang 2026-04-08 17:53:53 -07:00
parent 7654234f20
commit f93db822fd
No known key found for this signature in database
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
ui/litellm-dashboard/src/components/common_components/RouterSettingsAccordion.tsx
View file

@ -241,9 +241,13 @@ const RouterSettingsAccordion = forwardRef<RouterSettingsAccordionRef, RouterSet
key !== "fallbacks"
) {
const inputEl = document.querySelector(`input[name="${key}"]`) as HTMLInputElement | null;
if (inputEl && inputEl.value !== undefined && inputEl.value !== "") {
const parsed = parseInputValue(key, inputEl.value, value);
return [key, parsed];
if (inputEl) {
if (inputEl.value !== undefined && inputEl.value !== "") {
const parsed = parseInputValue(key, inputEl.value, value);
return [key, parsed];
}
// Input exists but is empty — user cleared it, set to null
return [key, null];
}
return [key, value];
} else if (key === "routing_strategy") {