From 5905100cdc4f56b3123b60765eca2df84a14a034 Mon Sep 17 00:00:00 2001 From: yuneng-jiang Date: Sat, 21 Mar 2026 00:35:36 -0700 Subject: [PATCH] address greptile review feedback (greploop iteration 3) - Fix double error handling in getTeamPermissionsCall: return empty data on HTTP error instead of calling handleError + throwing, preventing duplicate error notifications Co-Authored-By: Claude Opus 4.6 (1M context) --- ui/litellm-dashboard/src/components/networking.tsx | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/ui/litellm-dashboard/src/components/networking.tsx b/ui/litellm-dashboard/src/components/networking.tsx index 0ca66540822..d31e86a53b0 100644 --- a/ui/litellm-dashboard/src/components/networking.tsx +++ b/ui/litellm-dashboard/src/components/networking.tsx @@ -7269,12 +7269,11 @@ export const getTeamPermissionsCall = async (accessToken: string, teamId: string if (!response.ok) { const errorData = await response.json(); const errorMessage = deriveErrorMessage(errorData); - handleError(errorMessage); - throw new Error(errorMessage); + console.error("Available permissions fetch failed:", errorMessage); + return { all_available_permissions: [], team_member_permissions: [] }; } const data = await response.json(); - console.log("Team permissions response:", data); return data; } catch (error) { console.error("Failed to get team permissions:", error);