From 8b8a28848cb57811751ea9671c9214b1f30cfe56 Mon Sep 17 00:00:00 2001 From: Tin Chi Lo Date: Fri, 10 Jul 2026 16:09:42 -0700 Subject: [PATCH] fix(ui): drop the saved qualifier from the OAuth client field labels A plain optional label implies the field is persisted like any other form field; the qualifier was only there to contrast with the removed not-saved wording --- .../src/components/mcp_tools/PassthroughAuthorizeSection.tsx | 4 ++-- .../src/components/mcp_tools/create_mcp_server.test.tsx | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/ui/litellm-dashboard/src/components/mcp_tools/PassthroughAuthorizeSection.tsx b/ui/litellm-dashboard/src/components/mcp_tools/PassthroughAuthorizeSection.tsx index 09701f01535..727bc8804bd 100644 --- a/ui/litellm-dashboard/src/components/mcp_tools/PassthroughAuthorizeSection.tsx +++ b/ui/litellm-dashboard/src/components/mcp_tools/PassthroughAuthorizeSection.tsx @@ -45,7 +45,7 @@ export default function PassthroughAuthorizeSection({ authorize from the Tools page go through it.

OAuth Client ID (optional, saved)} + label={OAuth Client ID (optional)} name={["credentials", "client_id"]} extra="Only needed when the upstream does not support dynamic client registration (e.g. a pre-registered Slack app)." > @@ -55,7 +55,7 @@ export default function PassthroughAuthorizeSection({ /> OAuth Client Secret (optional, saved)} + label={OAuth Client Secret (optional)} name={["credentials", "client_secret"]} > { await waitFor(() => { expect(screen.getByRole("button", { name: "Authorize & Fetch Tools (browser-only)" })).toBeInTheDocument(); }); - expect(screen.getByText("OAuth Client ID (optional, saved)")).toBeInTheDocument(); - expect(screen.getByText("OAuth Client Secret (optional, saved)")).toBeInTheDocument(); + expect(screen.getByText("OAuth Client ID (optional)")).toBeInTheDocument(); + expect(screen.getByText("OAuth Client Secret (optional)")).toBeInTheDocument(); }, );