scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-11-12 08:42:18 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 207

fix(ui): preserve duration select callback shape

Browse source
This commit is contained in:
Yuneng Jiang 2026-08-12 14:11:21 -07:00
parent a32257008f
commit 78b36699df
No known key found for this signature in database
2 changed files with 16 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
ui/litellm-dashboard/src/components/common_components/DurationSelect.test.tsx
View file

@ -42,7 +42,7 @@ describe("DurationSelect", () => {
const dailyOption = dailyLabel.closest('[role="option"]') ?? dailyLabel;
await user.click(dailyOption);
expect(onChange.mock.calls.map(([selected]) => selected)).toContain("24h");
expect(onChange).toHaveBeenCalledWith("24h", expect.any(Object));
});
it("should accept and pass value prop to Select", () => {

21
ui/litellm-dashboard/src/components/common_components/DurationSelect.tsx
View file

@ -3,16 +3,23 @@ import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue } from "@
interface DurationSelectProps {
className?: string;
value?: string;
onChange?: (value: string) => void;
onChange?: (value: string, option: { value: string; label: string }) => void;
}
const DURATION_OPTIONS = [
{ value: "24h", label: "Daily" },
{ value: "7d", label: "Weekly" },
{ value: "30d", label: "Monthly" },
];
export default function DurationSelect({ className, value, onChange }: DurationSelectProps) {
return (
<Select
value={value}
onValueChange={(nextValue) => {
if (nextValue !== null) {
onChange?.(nextValue);
const selectedOption = DURATION_OPTIONS.find((option) => option.value === nextValue);
if (selectedOption) {
onChange?.(nextValue, selectedOption);
}
}}
>
@ -20,9 +27,11 @@ export default function DurationSelect({ className, value, onChange }: DurationS
<SelectValue placeholder="Select duration" />
</SelectTrigger>
<SelectContent>
<SelectItem value="24h">Daily</SelectItem>
<SelectItem value="7d">Weekly</SelectItem>
<SelectItem value="30d">Monthly</SelectItem>
{DURATION_OPTIONS.map((option) => (
<SelectItem key={option.value} value={option.value}>
{option.label}
</SelectItem>
))}
</SelectContent>
</Select>
);