Browse Source

fix(usage-logs): replace empty string with 'all' value in endpoint filter SelectItem

- Radix UI Select requires non-empty value prop for SelectItem
- Changed endpoint filter 'all endpoints' option from "" to "all"
- Updated value change handler to map "all" back to undefined
- Fixes: Uncaught Error - A <Select.Item /> must have a value prop that is not an empty string

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
ding113 4 months ago
parent
commit
66df9fcf5f

+ 3 - 3
src/app/[locale]/dashboard/logs/_components/usage-logs-filters.tsx

@@ -291,9 +291,9 @@ export function UsageLogsFilters({
         <div className="space-y-2">
           <Label>{t("logs.filters.endpoint")}</Label>
           <Select
-            value={localFilters.endpoint || ""}
+            value={localFilters.endpoint || "all"}
             onValueChange={(value: string) =>
-              setLocalFilters({ ...localFilters, endpoint: value || undefined })
+              setLocalFilters({ ...localFilters, endpoint: value === "all" ? undefined : value })
             }
             disabled={isEndpointLoading}
           >
@@ -309,7 +309,7 @@ export function UsageLogsFilters({
               />
             </SelectTrigger>
             <SelectContent>
-              <SelectItem value="">{t("logs.filters.allEndpoints")}</SelectItem>
+              <SelectItem value="all">{t("logs.filters.allEndpoints")}</SelectItem>
               {endpoints.map((endpoint) => (
                 <SelectItem key={endpoint} value={endpoint}>
                   {endpoint}