Refactor useEffect

src/App.tsx

@@ -41,15 +41,15 @@ function App() {
             <Route
               path="/onboarding-success"
               element={<OnboardingSuccess />}
-            ></Route>
+            />
             <Route
               path="/validator"
               element={<Validator />}
-            ></Route>
+            />
             <Route
               path="/validator-success"
               element={<ValidatorSuccess />}
-            ></Route>
+            />
           </Route>
           <Route path="*" element={<PageNotFound />} />
         </Routes>

src/components/TermsAndConditionsBox.tsx

@@ -16,6 +16,7 @@ const TermsAndConditionsBox = ({ height, onLoad }: TermsAndConditionsBoxProps )
 
   function onDocumentLoadSuccess({ numPages }: { numPages: number }): void {
     setNumPages(numPages);
+
     if (onLoad){
       onLoad();
     };

src/pages/ConnectWallet.tsx

@@ -15,25 +15,27 @@ const ConnectWallet = () => {
 
   const redirectTo = searchParams.get("redirectTo");
 
-  useEffect(()=>{
+  useEffect(() => {
     if (signClient && !session) {
       checkPersistedState(signClient);
     }
   }, [checkPersistedState, signClient, session]);
 
   useEffect(() => {
-    if (session) {
+    if (!session) {
-      if (redirectTo){
+      return;
-        navigate(`/${redirectTo}`, {
+    }
-          state: location.state
-        });
 
-      }
+    if (redirectTo) {
-      else {
+      navigate(`/${redirectTo}`, {
-        navigate("/sign-with-nitro-key", {
+        state: location.state
-          state: location.state
+      });
-        });
+    }
-      }
+
+    else {
+      navigate("/sign-with-nitro-key", {
+        state: location.state
+      });
     }
   }, [session, navigate, redirectTo, location.state]);
 

src/pages/SignWithNitroKey.tsx

@@ -24,7 +24,7 @@ const SignWithNitroKey = () => {
   const location = useLocation();
 
   useEffect(() => {
-    if (!session){
+    if (!session) {
       navigate("/connect-wallet?redirectTo=sign-with-nitro-key", {
         state: location.state,
       });