Merge pull request #15 from rzane/do-not-submit-twice

Avoid double submission
rzane · Aug 19, 2020 · 1711cf6 · 1711cf6
2 parents 23cf95e + 03ba59c
commit 1711cf6
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/src/useSubmit.ts b/src/useSubmit.ts
@@ -29,6 +29,10 @@ export function useSubmit<Value, Result>(
       event.stopPropagation();
     }
 
+    if (form.isSubmitting) {
+      return;
+    }
+
     form.setSubmitting(true);
 
     try {

diff --git a/test/useSubmit.test.ts b/test/useSubmit.test.ts
@@ -81,3 +81,11 @@ test("prevents default and stops propagation", async () => {
   expect(event.preventDefault).toHaveBeenCalled();
   expect(event.stopPropagation).toHaveBeenCalled();
 });
+
+test("makes no attempt to submit when already submitting", async () => {
+  const form: any = { isSubmitting: true };
+  const submit = jest.fn();
+  const onSubmit = renderHook(() => useSubmit(form, submit));
+  await onSubmit.result.current();
+  expect(submit).not.toHaveBeenCalled();
+});