Skip to content

Commit 1cfce58

Browse files
authored
fix: hide disabled buttons in hub under feature flag (#371)
1 parent 934831d commit 1cfce58

1 file changed

Lines changed: 22 additions & 12 deletions

File tree

src/Components/DashboardHub/DashboardTable/DashboardTable.tsx

Lines changed: 22 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ export const DashboardTable: React.FunctionComponent<DashboardTableProps> = ({ d
3737
const [dashboardToDelete, setDashboardToDelete] = useState<Dashboard | null>(null);
3838
const [duplicateDashboardId, setDuplicateDashboardId] = useState<number | null>(null);
3939
const isEnabledDelete = useFlag('platform.widget-layout.delete-dashboard');
40+
const isEnabledShare = useFlag('platform.widget-layout.share');
41+
const isEnabledEdit = useFlag('platform.widget-layout.edit_from_hub');
4042
const addNotification = useAddNotification();
4143
const setDefaultDashboard = useSetAtom(setDefaultDashboardAtom);
4244

@@ -113,12 +115,16 @@ export const DashboardTable: React.FunctionComponent<DashboardTableProps> = ({ d
113115

114116
// Row actions for each dashboard
115117
const getRowActions = (dashboard: Dashboard) => [
116-
{
117-
icon: <EditAltIcon />,
118-
title: 'Edit dashboard',
119-
isDisabled: true,
120-
onClick: () => console.log(`Edit dashboard ${dashboard.id}`),
121-
},
118+
...(isEnabledEdit
119+
? [
120+
{
121+
icon: <EditAltIcon />,
122+
title: 'Edit dashboard',
123+
isDisabled: true,
124+
onClick: () => console.log(`Edit dashboard ${dashboard.id}`),
125+
},
126+
]
127+
: []),
122128
{
123129
icon: <HomeIcon />,
124130
title: 'Set as homepage',
@@ -131,12 +137,16 @@ export const DashboardTable: React.FunctionComponent<DashboardTableProps> = ({ d
131137
title: 'Copy configuration string',
132138
onClick: () => handleCopyConfiguration(dashboard),
133139
},
134-
{
135-
icon: <UsersIcon />,
136-
title: 'Share dashboard',
137-
isDisabled: true,
138-
onClick: () => console.log(`Share dashboard ${dashboard.id}`),
139-
},
140+
...(isEnabledShare
141+
? [
142+
{
143+
icon: <UsersIcon />,
144+
title: 'Share dashboard',
145+
isDisabled: true,
146+
onClick: () => console.log(`Share dashboard ${dashboard.id}`),
147+
},
148+
]
149+
: []),
140150
...(isEnabledDelete
141151
? [
142152
{

0 commit comments

Comments
 (0)