Merge pull request #241 from janong24/signUpBuxFix

added delay for getUser in order for it to run after createUser

Author: MrMelon1232

client/app/contexts/UserContext.tsx

@@ -1,19 +1,19 @@
-'use client';
+"use client";
 import {
   createContext,
   useContext,
   FC,
   ReactNode,
   useState,
   useEffect,
-} from 'react';
-import getUser from '@/app/http/getUser';
-import updateUser from '@/app/http/updateUser';
-import { useAuth } from './AuthContext';
-import { useRouter } from 'next/navigation';
-import { auth } from '@/app/config/firebase';
-import { signOut } from 'firebase/auth';
-import {useProp} from '@/app/contexts/PropContext';
+} from "react";
+import getUser from "@/app/http/getUser";
+import updateUser from "@/app/http/updateUser";
+import { useAuth } from "./AuthContext";
+import { useRouter } from "next/navigation";
+import { auth } from "@/app/config/firebase";
+import { signOut } from "firebase/auth";
+import { useProp } from "@/app/contexts/PropContext";
 type UserAttributes = {
   id: number;
   uid: string;
@@ -38,13 +38,13 @@ interface UserContextProps {
   updateCurrentUser: (userData: EditableUserAttributes) => void;
 }
 
-const logger = require('../../logger');
+const logger = require("../../logger");
 const UserContext = createContext<UserContextProps | undefined>(undefined);
 
 export const useUser = () => {
   const context = useContext(UserContext);
   if (context === undefined) {
-    throw new Error('useUser must be used within a UserProvider');
+    throw new Error("useUser must be used within a UserProvider");
   }
   return context;
 };
@@ -59,41 +59,43 @@ export const UserProvider: FC<UserProviderProps> = ({ children }) => {
   // can remove the following line as we check for user in req. maybe no more use for user const too.
   const uid = user ? user.uid : null; // Access the UID if the user is authenticated
   const router = useRouter();
-  const {handlePopUp, loading, handleLoading} = useProp();
+  const { handlePopUp, loading, handleLoading } = useProp();
   useEffect(() => {
     const fetchUserData = () => {
-      handleLoading(true)
+      handleLoading(true);
       if (uid) {
         getUser()
           .then((userData) => {
             setUserInfo(userData);
-            router.push('/tpage');
-                handleLoading(false,1000)
+            router.push("/tpage");
+            handleLoading(false, 1000);
           })
           .catch((error) => {
-           handlePopUp('error',error.message);
+            handlePopUp("error", error.message);
             signOut(auth);
           });
       } else {
         setUserInfo(null);
-        handleLoading(false)
+        handleLoading(false);
       }
     };
     if (user) {
-      fetchUserData();
+      setTimeout(() => {
+        fetchUserData();
+      }, 1000);
     } else {
-      handleLoading(false)
+      handleLoading(false);
     }
   }, [uid]);
   const updateCurrentUser = (userData: EditableUserAttributes) => {
     if (uid) {
       updateUser(userData)
         .then((response) => {
-          logger.info('User updated successfully:', response);
+          logger.info("User updated successfully:", response);
           setUserInfo(response.data[1]);
         })
         .catch((error) => {
-         logger.error('Error updating user:', error)
+          logger.error("Error updating user:", error);
         });
     }
   };
@@ -102,9 +104,5 @@ export const UserProvider: FC<UserProviderProps> = ({ children }) => {
     updateCurrentUser,
   };
 
-  return (
-    <UserContext.Provider value={value}>
-      {children}
-    </UserContext.Provider>
-  );
+  return <UserContext.Provider value={value}>{children}</UserContext.Provider>;
 };