fix: match registerPluginSettings name to deployed plugin + fix badge nav URL
The settings component never rendered in E2E because registerPluginSettings used 'headlamp-polaris' (and before that 'headlamp-polaris-plugin'), but the deployed plugin directory is static-plugins/polaris — Headlamp identifies it as 'polaris'. Changed to 'polaris' so the settings page can match the registration to the correct plugin. The badge click navigated to /polaris instead of /c/main/polaris because Router.createRouteURL did not reliably resolve the cluster prefix from the plugin context. Replaced with direct URL construction using K8s.useCluster(). Co-Authored-By: Paperclip <noreply@paperclip.ing>
This commit is contained in:
@@ -1,4 +1,4 @@
|
||||
import { K8s, Router } from '@kinvolk/headlamp-plugin/lib';
|
||||
import { K8s } from '@kinvolk/headlamp-plugin/lib';
|
||||
import { useTheme } from '@mui/material/styles';
|
||||
import React from 'react';
|
||||
import { useHistory } from 'react-router-dom';
|
||||
@@ -36,7 +36,8 @@ export default function AppBarScoreBadge() {
|
||||
};
|
||||
|
||||
const handleClick = () => {
|
||||
history.push(Router.createRouteURL('polaris', { cluster: cluster ?? '' }));
|
||||
const prefix = cluster ? `/c/${cluster}` : '';
|
||||
history.push(`${prefix}/polaris`);
|
||||
};
|
||||
|
||||
return (
|
||||
|
||||
+1
-1
@@ -99,7 +99,7 @@ registerRoute({
|
||||
});
|
||||
|
||||
// Register plugin settings
|
||||
registerPluginSettings('headlamp-polaris', PolarisSettings, true);
|
||||
registerPluginSettings('polaris', PolarisSettings, true);
|
||||
|
||||
// Register details view section for supported controller types
|
||||
registerDetailsViewSection(({ resource }) => {
|
||||
|
||||
Reference in New Issue
Block a user