From 2a0ab66f2ba6ded0cf07d89ab03acbef782a4ede Mon Sep 17 00:00:00 2001 From: mkkk997750 Date: Wed, 15 Apr 2026 18:02:37 +0530 Subject: [PATCH] fix: replace loose equality operators with strict equality in UI dashboard --- .../models-and-endpoints/ModelsAndEndpointsView.tsx | 4 ++-- .../models-and-endpoints/components/AllModelsTab.tsx | 2 +- ui/litellm-dashboard/src/app/(dashboard)/networking.ts | 2 +- .../src/app/(dashboard)/teams/TeamsView.tsx | 6 +++--- .../(dashboard)/teams/components/TeamsTable/TeamsTable.tsx | 2 +- .../(dashboard)/teams/components/modals/CreateTeamModal.tsx | 2 +- 6 files changed, 9 insertions(+), 9 deletions(-) diff --git a/ui/litellm-dashboard/src/app/(dashboard)/models-and-endpoints/ModelsAndEndpointsView.tsx b/ui/litellm-dashboard/src/app/(dashboard)/models-and-endpoints/ModelsAndEndpointsView.tsx index 514ae673d06..bdee2ae3425 100644 --- a/ui/litellm-dashboard/src/app/(dashboard)/models-and-endpoints/ModelsAndEndpointsView.tsx +++ b/ui/litellm-dashboard/src/app/(dashboard)/models-and-endpoints/ModelsAndEndpointsView.tsx @@ -113,7 +113,7 @@ const ModelsAndEndpointsView: React.FC = ({ premiumUser, te const getProviderFromModel = (model: string) => { if (modelCostMapData !== null && modelCostMapData !== undefined) { - if (typeof modelCostMapData == "object" && model in modelCostMapData) { + if (typeof modelCostMapData === "object" && model in modelCostMapData) { return modelCostMapData[model]["litellm_provider"]; } } @@ -229,7 +229,7 @@ const ModelsAndEndpointsView: React.FC = ({ premiumUser, te const isLoading = isLoadingModels || isLoadingModelCostMap || isLoadingCredentials || isLoadingUISettings; - if (userRole && userRole == "Admin Viewer") { + if (userRole && userRole === "Admin Viewer") { const { Title, Paragraph } = Typography; return (
diff --git a/ui/litellm-dashboard/src/app/(dashboard)/models-and-endpoints/components/AllModelsTab.tsx b/ui/litellm-dashboard/src/app/(dashboard)/models-and-endpoints/components/AllModelsTab.tsx index 5431c196883..9034e11b56a 100644 --- a/ui/litellm-dashboard/src/app/(dashboard)/models-and-endpoints/components/AllModelsTab.tsx +++ b/ui/litellm-dashboard/src/app/(dashboard)/models-and-endpoints/components/AllModelsTab.tsx @@ -113,7 +113,7 @@ const AllModelsTab = ({ const getProviderFromModel = (model: string) => { if (modelCostMapData !== null && modelCostMapData !== undefined) { - if (typeof modelCostMapData == "object" && model in modelCostMapData) { + if (typeof modelCostMapData === "object" && model in modelCostMapData) { return modelCostMapData[model]["litellm_provider"]; } } diff --git a/ui/litellm-dashboard/src/app/(dashboard)/networking.ts b/ui/litellm-dashboard/src/app/(dashboard)/networking.ts index 7fb09d61a5d..de401a92242 100644 --- a/ui/litellm-dashboard/src/app/(dashboard)/networking.ts +++ b/ui/litellm-dashboard/src/app/(dashboard)/networking.ts @@ -7,7 +7,7 @@ export const fetchTeams = async ( currentOrg: Organization | null, ) => { let givenTeams; - if (userRole != "Admin" && userRole != "Admin Viewer") { + if (userRole !== "Admin" && userRole !== "Admin Viewer") { givenTeams = await teamListCall(accessToken, currentOrg?.organization_id || null, userID); } else { givenTeams = await teamListCall(accessToken, currentOrg?.organization_id || null); diff --git a/ui/litellm-dashboard/src/app/(dashboard)/teams/TeamsView.tsx b/ui/litellm-dashboard/src/app/(dashboard)/teams/TeamsView.tsx index fcad42d3a75..6c1af524add 100644 --- a/ui/litellm-dashboard/src/app/(dashboard)/teams/TeamsView.tsx +++ b/ui/litellm-dashboard/src/app/(dashboard)/teams/TeamsView.tsx @@ -245,7 +245,7 @@ const TeamsView: React.FC = ({
- {(userRole == "Admin" || userRole == "Org Admin") && ( + {(userRole === "Admin" || userRole === "Org Admin") && ( @@ -277,7 +277,7 @@ const TeamsView: React.FC = ({ }} accessToken={accessToken} is_team_admin={is_team_admin(teams?.find((team) => team.team_id === selectedTeamId))} - is_proxy_admin={userRole == "Admin"} + is_proxy_admin={userRole === "Admin"} is_org_admin={(() => { const team = teams?.find((t) => t.team_id === selectedTeamId); if (!team?.organization_id || !organizations || !userID) return false; @@ -341,7 +341,7 @@ const TeamsView: React.FC = ({ )} )} - {(userRole == "Admin" || userRole == "Org Admin") && ( + {(userRole === "Admin" || userRole === "Org Admin") && ( - {userRole == "Admin" ? ( + {userRole === "Admin" ? ( <> ) => { try { console.log(`formValues: ${JSON.stringify(formValues)}`); - if (accessToken != null) { + if (accessToken !== null) { const newTeamAlias = formValues?.team_alias; const existingTeamAliases = teams?.map((t) => t.team_alias) ?? []; let organizationId = formValues?.organization_id || currentOrg?.organization_id;