Skip to content

Commit 8838efd

Browse files
authored
fix: Bulk approve was always a merge (#363)
closes Visual-Regression-Tracker/Visual-Regression-Tracker#437
1 parent 0cd8d6d commit 8838efd

File tree

1 file changed

+2
-7
lines changed

1 file changed

+2
-7
lines changed

src/components/TestRunList/BulkOperation.tsx

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -47,10 +47,7 @@ export const BulkOperation: React.FunctionComponent = () => {
4747
);
4848

4949
const isMerge: boolean = React.useMemo(
50-
() =>
51-
!!rows.find((row: GridRowModel) =>
52-
selectedIds.includes(row.id.toString()),
53-
),
50+
() => !!rows.find((row: GridRowModel) => selectedIds.includes(row.id.toString()))?.model.merge,
5451
[selectedIds, rows],
5552
);
5653

@@ -271,9 +268,7 @@ export const BulkOperation: React.FunctionComponent = () => {
271268
onCancel={dismissDialog}
272269
content={
273270
<Typography>
274-
{`Are you sure you want to ${submitButtonText().toLowerCase()} ${
275-
selectedRows.length
276-
} items?`}
271+
{`Are you sure you want to ${submitButtonText().toLowerCase()} ${selectedRows.length} items?`}
277272
</Typography>
278273
}
279274
onSubmit={() => {

0 commit comments

Comments
 (0)