Refactor ProxyMenu component to simplify conditional rendering of proxy start and stop options

caffeinated92 · caffeinated92 · commit 8590a5862c0a · 2025-04-18T09:55:59.000Z
diff --git a/share/dashboard_react/src/Pages/Dashboard/components/Proxies/ProxyMenu.jsx b/share/dashboard_react/src/Pages/Dashboard/components/Proxies/ProxyMenu.jsx
@@ -78,8 +78,7 @@ function ProxyMenu({ clusterName, row, isDesktop, colorScheme, from = 'tableView
                 }
               ]
             : []),
-          ...(user?.grants['proxy-start'] && isMenuOptionsVisible
-            ? [
+          ...(user?.grants['proxy-start'] ? [
                 {
                   name: 'Start Proxy',
                   onClick: () => {
@@ -90,8 +89,7 @@ function ProxyMenu({ clusterName, row, isDesktop, colorScheme, from = 'tableView
                 }
               ]
             : []),
-          ...(user?.grants['proxy-stop'] && isMenuOptionsVisible
-            ? [
+          ...(user?.grants['proxy-stop'] ? [
                 {
                   name: 'Stop Proxy',
                   onClick: () => {

Original file line number	Diff line number	Diff line change
`@@ -78,8 +78,7 @@ function ProxyMenu({ clusterName, row, isDesktop, colorScheme, from = 'tableView`
`78`	`78`	`}`
`79`	`79`	`]`
`80`	`80`	`: []),`
`81`		`- ...(user?.grants['proxy-start'] && isMenuOptionsVisible`
`82`		`- ? [`
	`81`	`+ ...(user?.grants['proxy-start'] ? [`
`83`	`82`	`{`
`84`	`83`	`name: 'Start Proxy',`
`85`	`84`	`onClick: () => {`
`@@ -90,8 +89,7 @@ function ProxyMenu({ clusterName, row, isDesktop, colorScheme, from = 'tableView`
`90`	`89`	`}`
`91`	`90`	`]`
`92`	`91`	`: []),`
`93`		`- ...(user?.grants['proxy-stop'] && isMenuOptionsVisible`
`94`		`- ? [`
	`92`	`+ ...(user?.grants['proxy-stop'] ? [`
`95`	`93`	`{`
`96`	`94`	`name: 'Stop Proxy',`
`97`	`95`	`onClick: () => {`