fix(user): prevent early form submitting

jwplayer · Aug 5, 2021 · a405ebe · a405ebe
1 parent b31a82a
commit a405ebe
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/components/Form/Form.tsx b/src/components/Form/Form.tsx
@@ -53,15 +53,18 @@ const Form = ({ initialValues, editing = true, children, onSubmit }: Props): JSX
   };
 
   useEffect(() => {
-    // todo: only update new key/values?
     setValues(initialValues);
   }, [initialValues]);
 
   if (!editing) {
     return children({ values });
   }
 
-  return <form noValidate>{children({ values, handleChange, handleSubmit, handleReset, hasChanged })}</form>;
+  return (
+    <form noValidate onSubmit={(event) => event.preventDefault()}>
+      {children({ values, handleChange, handleSubmit, handleReset, hasChanged })}
+    </form>
+  );
 };
 
 export default Form;