Skip to content

Commit 1427d26

Browse files
rhamiltoclaude
andcommitted
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 <noreply@anthropic.com>
1 parent 2489c9a commit 1427d26

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

frontend/packages/helm-plugin/src/components/details-page/HelmReleaseDetailsPage.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,9 @@ const HelmReleaseDetailsPage: FC = () => {
1313
const handleNamespaceChange = useCallback(
1414
(newNamespace: string): void => {
1515
if (newNamespace === ALL_NAMESPACES_KEY) {
16-
navigate('/helm-releases/all-namespaces');
16+
navigate('/helm/all-namespaces');
1717
} else {
18-
navigate(`/helm-releases/ns/${newNamespace}`);
18+
navigate(`/helm/ns/${newNamespace}`);
1919
}
2020
},
2121
[navigate],

frontend/packages/helm-plugin/src/utils/helm-utils.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -222,11 +222,11 @@ export const getOriginRedirectURL = (
222222
case HelmActionOrigins.topology:
223223
return `/topology/ns/${namespace}`;
224224
case HelmActionOrigins.list:
225-
return `/helm-releases/ns/${namespace}`;
225+
return `/helm/ns/${namespace}`;
226226
case HelmActionOrigins.details:
227227
return `/helm-releases/ns/${namespace}/release/${releaseName}`;
228228
default:
229-
return `/helm-releases/ns/${namespace}`;
229+
return `/helm/ns/${namespace}`;
230230
}
231231
};
232232

0 commit comments

Comments
 (0)