Merge pull request #68 from DouglasNeuroInformatics/dev

fix: allow params in destructive action

Author: joshunrau

src/hooks/useDestructiveAction/useDestructiveAction.ts

@@ -1,12 +1,15 @@
 import { useCallback } from 'react';
 
-import { useDestructiveActionStore } from './useDestructiveActionStore';
+import type { Promisable } from 'type-fest';
 
-import type { DestructiveAction } from './useDestructiveActionStore';
+import { useDestructiveActionStore } from './useDestructiveActionStore';
 
-export function useDestructiveAction(destructiveAction: DestructiveAction) {
+export function useDestructiveAction<TArgs extends any[]>(destructiveAction: (...args: TArgs) => Promisable<void>) {
   const addPendingDestructiveAction = useDestructiveActionStore((store) => store.addPendingDestructiveAction);
-  return useCallback(() => {
-    addPendingDestructiveAction(destructiveAction);
-  }, [destructiveAction, addPendingDestructiveAction]);
+  return useCallback(
+    (...args: TArgs) => {
+      addPendingDestructiveAction(() => destructiveAction(...args));
+    },
+    [destructiveAction, addPendingDestructiveAction]
+  );
 }

src/providers/CoreProvider/CoreProvider.stories.tsx

@@ -10,7 +10,9 @@ type Story = StoryObj<typeof CoreProvider>;
 
 const Children = () => {
   const addNotification = useNotificationsStore((store) => store.addNotification);
-  const destructiveAction = useDestructiveAction(() => alert('Delete'));
+  const destructiveAction = useDestructiveAction((event: React.MouseEvent<HTMLButtonElement>) => {
+    alert(`Delete at Event Time: ${event.timeStamp}`);
+  });
   return (
     <div>
       <Button