From 273832e9dc029b87ce7d952047d7fb05dfb69c41 Mon Sep 17 00:00:00 2001 From: Falko Woudstra Date: Sun, 21 Jul 2024 22:21:47 +0200 Subject: [PATCH] Update NavigationModal to support multiple ConfirmationModals and remove handleDeleteClick from useRecordActionBar. --- .../record-action-bar/hooks/useRecordActionBar.tsx | 1 - .../ui/navigation/shared/components/NavigationModal.tsx | 6 +++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/packages/twenty-front/src/modules/object-record/record-action-bar/hooks/useRecordActionBar.tsx b/packages/twenty-front/src/modules/object-record/record-action-bar/hooks/useRecordActionBar.tsx index 490467dae757..9765a085f5af 100644 --- a/packages/twenty-front/src/modules/object-record/record-action-bar/hooks/useRecordActionBar.tsx +++ b/packages/twenty-front/src/modules/object-record/record-action-bar/hooks/useRecordActionBar.tsx @@ -136,7 +136,6 @@ export const useRecordActionBar = ({ accent: 'danger', onClick: () => { setIsDeleteRecordsModalOpen(true); - handleDeleteClick(); }, ConfirmationModal: ( { return (
- {actionBarEntries[0]?.ConfirmationModal ?? null} + {actionBarEntries.map((actionBarEntry, index) => + actionBarEntry.ConfirmationModal ? ( +
{actionBarEntry.ConfirmationModal}
+ ) : null, + )}
); };