Merge pull request #46 from AthulJoshi/shareinvite-update

minor fixes
anandbaburajan · Jan 30, 2021 · 87382d0 · 87382d0 · vercel · Jan 30, 2021
2 parents f3c68e8 + 432187e
commit 87382d0
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 7 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/components/shareinvite.tsx b/src/components/shareinvite.tsx
@@ -65,7 +65,7 @@ const Invitation = (props: {
       evt.preventDefault();
       let value = currentEmail.trim();
       if (value && isValid(value)) {
-        setEmails([...emailList, currentEmail]);
+        setEmails([...emailList, value]);
         setCurrentEmail("");
       }
     }
@@ -87,13 +87,17 @@ const Invitation = (props: {
       setEmails([...emailList, ...toBeAdded]);
     }
   };
+  const handlePreSubmit = (): void => {
+    let value = currentEmail.trim();
+    if (value && isValid(value)) {
+      setEmails([...emailList, value]);
+      setCurrentEmail("");
+    }
+    handleSubmit();
+  }
   /* added void below( remove this comment at last PR) */
   const handleSubmit = async (): Promise<void> => {
     /* console.log(emailList); which is also to be removed */
-    if(isValid(currentEmail)) {
-      setEmails([...emailList, currentEmail]);
-      setCurrentEmail("");
-    }
     const mailerArgs: MailerPollArgs = {
       pollID: pollid,
       pollTitle: polltitle,
@@ -102,7 +106,7 @@ const Invitation = (props: {
       senderEmailID: loggedInUserEmailID,
     };
     const { statusCode } = await mailerAPI.sendPollInvites(mailerArgs, token);
-    if (statusCode !== 404) {
+    if (statusCode === 200) {
       props.onChangeS(true);
     } else {
       props.onChangeF(true);
@@ -149,7 +153,7 @@ const Invitation = (props: {
             onChange={handleChange}
             onPaste={handlePaste}
           />
-          <Button className="my-2" variant="light" onClick={handleSubmit}>
+          <Button className="my-2" variant="light" onClick={handlePreSubmit}>
             Invite
           </Button>
         </Form.Group>