From 8838efd9930d28cd263d4f01895dca860eacf63f Mon Sep 17 00:00:00 2001 From: Pavel Strunkin Date: Fri, 26 Jul 2024 19:56:07 +0300 Subject: [PATCH] fix: Bulk approve was always a merge (#363) closes https://github.com/Visual-Regression-Tracker/Visual-Regression-Tracker/issues/437 --- src/components/TestRunList/BulkOperation.tsx | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/components/TestRunList/BulkOperation.tsx b/src/components/TestRunList/BulkOperation.tsx index ef8248d..974689d 100644 --- a/src/components/TestRunList/BulkOperation.tsx +++ b/src/components/TestRunList/BulkOperation.tsx @@ -47,10 +47,7 @@ export const BulkOperation: React.FunctionComponent = () => { ); const isMerge: boolean = React.useMemo( - () => - !!rows.find((row: GridRowModel) => - selectedIds.includes(row.id.toString()), - ), + () => !!rows.find((row: GridRowModel) => selectedIds.includes(row.id.toString()))?.model.merge, [selectedIds, rows], ); @@ -271,9 +268,7 @@ export const BulkOperation: React.FunctionComponent = () => { onCancel={dismissDialog} content={ - {`Are you sure you want to ${submitButtonText().toLowerCase()} ${ - selectedRows.length - } items?`} + {`Are you sure you want to ${submitButtonText().toLowerCase()} ${selectedRows.length} items?`} } onSubmit={() => {