Bläddra i källkod

fix: remove form reset logic from useEffect in task import

satnaing 1 år sedan
förälder
incheckning
e53de3ad5e
1 ändrade filer med 7 tillägg och 9 borttagningar
  1. 7 9
      src/features/tasks/components/tasks-import-dialog.tsx

+ 7 - 9
src/features/tasks/components/tasks-import-dialog.tsx

@@ -1,4 +1,3 @@
-import { useEffect } from 'react'
 import { z } from 'zod'
 import { useForm } from 'react-hook-form'
 import { zodResolver } from '@hookform/resolvers/zod'
@@ -71,15 +70,14 @@ export function TasksImportDialog({ open, onOpenChange }: Props) {
     onOpenChange(false)
   }
 
-  /* Reset form when modal is closed */
-  useEffect(() => {
-    if (!open) {
-      form.reset()
-    }
-  }, [form, open])
-
   return (
-    <Dialog open={open} onOpenChange={onOpenChange}>
+    <Dialog
+      open={open}
+      onOpenChange={(val) => {
+        onOpenChange(val)
+        form.reset()
+      }}
+    >
       <DialogContent className='sm:max-w-sm gap-2'>
         <DialogHeader>
           <DialogTitle>Import Tasks</DialogTitle>