Merge pull request #121 from meshery/theBeginner86-patch-1

[bug] Fix multiple form submission
meshery · Jul 4, 2023 · 9abad6c · 9abad6c
2 parents 3bafe87 + 0b763ad
commit 9abad6c
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/site/src/components/SignupForm/index.js b/site/src/components/SignupForm/index.js
@@ -5,7 +5,7 @@ import MesheryText from "../../assets/images/meshery-light-text.svg";
 import SignupFormWrapper from "./signupform.style.js";
 import DiscussCallout from "../Discuss-Callout";
 
-const SignupForm = (props) => {
+const SignupForm = ({onSubmit}) => {
   // Form values
   const [memberFormOne, setMemberFormOne] = useState({});
 
@@ -29,7 +29,7 @@ const SignupForm = (props) => {
 
   useEffect(() => {
     if (formSubmitted) {
-      props.onSubmit();
+      onSubmit();
 
       axios.post("https://hook.us1.make.com/x4crqi16j8wfhctosk8y47fj6gknyvvh", {
         memberFormOne
@@ -40,7 +40,7 @@ const SignupForm = (props) => {
         behavior: "smooth"
       });
     }
-  }, [formSubmitted, memberFormOne, props]);
+  }, [formSubmitted, memberFormOne]);
 
   const PlayFormComponent = () => {
     return (