mobile: SignIn: use promise resolution instead of try-catch

Purdue-CS307-tsundoku · Feb 25, 2024 · dddf409 · dddf409
1 parent 83d4e2b
commit dddf409
Showing 1 changed file with 26 additions and 15 deletions.
diff --git a/mobile/Screens/SignIn.tsx b/mobile/Screens/SignIn.tsx
@@ -36,25 +36,36 @@ export const SignIn = (props) => {
     Keyboard.dismiss();
   };
   const handleSignIn = () => {
-    try {
-      account.createEmailSession(email, password).then((session) => {
+    account
+      .createEmailSession(email, password)
+      .then((session) => {
         setSession(session);
 
-        account.get().then((user) => {
-          setLoggedInUser(user);
+        account
+          .get()
+          .then((user) => {
+            setLoggedInUser(user);
 
-          // if sign-in successful, nav to next screen
-          navigation.navigate("navbar");
-        });
+            // if sign-in successful, nav to next screen
+            navigation.navigate("navbar");
+          })
+          .catch((error: any) => {
+            if (error instanceof Error) {
+              setErrorMessage(error.message);
+              setErrorModalVisible(true);
+            } else {
+              throw error;
+            }
+          });
+      })
+      .catch((error: any) => {
+        if (error instanceof Error) {
+          setErrorMessage(error.message);
+          setErrorModalVisible(true);
+        } else {
+          throw error;
+        }
       });
-    } catch (error: any) {
-      if (error instanceof Error) {
-        setErrorMessage(error.message);
-        setErrorModalVisible(true);
-      } else {
-        throw error;
-      }
-    }
   };
 
   return (