feat: onChange can handle undefined event

thomasskk · Sep 17, 2022 · 788d039 · 788d039
1 parent 13f89ef
commit 788d039
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/react-pocket-form/src/types/index.ts b/packages/react-pocket-form/src/types/index.ts
@@ -91,7 +91,7 @@ export type UseFormRegisterOptions<T = any, P extends Path<T> = any> = {
   validateOn?: 'change' | 'blur' | 'submit'
   revalidateOn?: 'change' | 'submit' | 'blur'
   onChange?: (
-    element: Element,
+    element?: Element,
     value?: PropertyType<T, P>
   ) => void | Promise<void>
   onBlur?: (
@@ -117,7 +117,7 @@ export type UseFormRegister<T = any> = <P extends Path<T>>(
 ) => UseFormRegisterReturn<T, P>
 
 export type UseFormRegisterReturn<T = any, P extends Path<T> = any> = {
-  onChange: (event: eventEl) => Promise<void> | void
+  onChange: (event?: eventEl) => Promise<void> | void
   onBlur: (event: eventEl) => Promise<void> | void
   ref: (el: Element, overrideValue?: PropertyType<T, P>) => void
   name: string

diff --git a/packages/react-pocket-form/src/useForm.ts b/packages/react-pocket-form/src/useForm.ts
@@ -262,7 +262,7 @@ export function useForm<T extends object = any>({
 
         watchStore.get(name)?.()
 
-        await onChange?.(event.currentTarget, getValue(name))
+        await onChange?.(event?.currentTarget, getValue(name))
       },
       ref: async (element, overrideValue) => {
         const refValue = refStore.get(name)