Skip to content

Commit 43cd5e9

Browse files
sjd78mguetta1
andauthored
✨ Set application delete as danger kebab menu item (#1829) (#1840)
Part of #1318 Resolves: https://issues.redhat.com/browse/MTA-2261 Backport-of: #1829 Signed-off-by: Maayan Hadasi <[email protected]> Signed-off-by: Scott J Dickerson <[email protected]> Co-authored-by: Maayan Hadasi <[email protected]>
1 parent 7d77ba3 commit 43cd5e9

File tree

1 file changed

+22
-20
lines changed

1 file changed

+22
-20
lines changed

client/src/app/pages/applications/applications-table/applications-table.tsx

+22-20
Original file line numberDiff line numberDiff line change
@@ -1010,26 +1010,6 @@ export const ApplicationsTable: React.FC = () => {
10101010
},
10111011
]
10121012
: []),
1013-
...(applicationWriteAccess
1014-
? [
1015-
{
1016-
isAriaDisabled:
1017-
application.migrationWave !== null,
1018-
tooltipProps: {
1019-
content:
1020-
application.migrationWave !== null
1021-
? t(
1022-
"message.cannotDeleteApplicationsAssignedToMigrationWave"
1023-
)
1024-
: "",
1025-
},
1026-
1027-
title: t("actions.delete"),
1028-
onClick: () =>
1029-
setApplicationsToDelete([application]),
1030-
},
1031-
]
1032-
: []),
10331013
...(dependenciesWriteAccess
10341014
? [
10351015
{
@@ -1075,6 +1055,28 @@ export const ApplicationsTable: React.FC = () => {
10751055
},
10761056
]
10771057
: []),
1058+
...(applicationWriteAccess
1059+
? [
1060+
{ isSeparator: true },
1061+
{
1062+
isAriaDisabled:
1063+
application.migrationWave !== null,
1064+
tooltipProps: {
1065+
content:
1066+
application.migrationWave !== null
1067+
? t(
1068+
"message.cannotDeleteApplicationsAssignedToMigrationWave"
1069+
)
1070+
: "",
1071+
},
1072+
1073+
title: t("actions.delete"),
1074+
onClick: () =>
1075+
setApplicationsToDelete([application]),
1076+
isDanger: true,
1077+
},
1078+
]
1079+
: []),
10781080
]}
10791081
/>
10801082
</Td>

0 commit comments

Comments
 (0)