From c446ef8e8ba4f1b3d9fe05f04a44547605475543 Mon Sep 17 00:00:00 2001 From: Robb Hamilton Date: Thu, 23 Apr 2026 17:11:46 -0400 Subject: [PATCH] OCPBUGS-84296: fix Helm list page redirect after upgrade/rollback The Helm list page is at /helm/ns/{namespace} for all perspectives. There is no /helm-releases/ns/{namespace} route - only detail and form routes use the /helm-releases prefix. Updated getOriginRedirectURL to always redirect to /helm/ns/{namespace} for list page, and fixed HelmReleaseDetailsPage namespace change handler to use the correct route. Co-Authored-By: Claude Sonnet 4.5 --- .../src/components/details-page/HelmReleaseDetailsPage.tsx | 4 ++-- frontend/packages/helm-plugin/src/utils/helm-utils.ts | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/frontend/packages/helm-plugin/src/components/details-page/HelmReleaseDetailsPage.tsx b/frontend/packages/helm-plugin/src/components/details-page/HelmReleaseDetailsPage.tsx index c74aa148e80..41deb482fd0 100644 --- a/frontend/packages/helm-plugin/src/components/details-page/HelmReleaseDetailsPage.tsx +++ b/frontend/packages/helm-plugin/src/components/details-page/HelmReleaseDetailsPage.tsx @@ -13,9 +13,9 @@ const HelmReleaseDetailsPage: FC = () => { const handleNamespaceChange = useCallback( (newNamespace: string): void => { if (newNamespace === ALL_NAMESPACES_KEY) { - navigate('/helm-releases/all-namespaces'); + navigate('/helm/all-namespaces'); } else { - navigate(`/helm-releases/ns/${newNamespace}`); + navigate(`/helm/ns/${newNamespace}`); } }, [navigate], diff --git a/frontend/packages/helm-plugin/src/utils/helm-utils.ts b/frontend/packages/helm-plugin/src/utils/helm-utils.ts index 2675dcabf65..34814f31c00 100644 --- a/frontend/packages/helm-plugin/src/utils/helm-utils.ts +++ b/frontend/packages/helm-plugin/src/utils/helm-utils.ts @@ -222,11 +222,11 @@ export const getOriginRedirectURL = ( case HelmActionOrigins.topology: return `/topology/ns/${namespace}`; case HelmActionOrigins.list: - return `/helm-releases/ns/${namespace}`; + return `/helm/ns/${namespace}`; case HelmActionOrigins.details: return `/helm-releases/ns/${namespace}/release/${releaseName}`; default: - return `/helm-releases/ns/${namespace}`; + return `/helm/ns/${namespace}`; } };