Merge staging - Remove welcome roll condition (#47)

* removed welcome roll condition (#46)
BuidlGuidl · Jul 24, 2024 · 8be238a · 8be238a
1 parent 4a481d6
commit 8be238a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/packages/nextjs/components/dicedemo/GameJoinForm.tsx b/packages/nextjs/components/dicedemo/GameJoinForm.tsx
@@ -73,7 +73,7 @@ const GameJoinForm = ({
   // };
 
   useEffect(() => {
-    if (invite) {
+    if (invite && playerAddress) {
       handleJoinGame(invite as string);
     }
     if (Object.keys(router.query).length > 0) {
@@ -83,7 +83,7 @@ const GameJoinForm = ({
       });
     }
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [invite]);
+  }, [invite, playerAddress]);
 
   return (
     <div className="w-full">

diff --git a/packages/nextjs/pages/index.tsx b/packages/nextjs/pages/index.tsx
@@ -12,13 +12,11 @@ const Home: NextPage = () => {
   const { invite } = router.query;
   const [gameState, setGameState] = useState<"createGame" | "joinGame">("joinGame");
   const [inviteCode, setInviteCode] = useState("");
-  const [showWelcomeRoll, setShowWelcomRoll] = useState(true);
 
   useEffect(() => {
     if (invite) {
       setGameState("joinGame");
       setInviteCode(invite as string);
-      setShowWelcomRoll(false);
     }
   }, [invite]);
 
@@ -36,7 +34,7 @@ const Home: NextPage = () => {
 
         <div>
           <div className="mx-auto mt-5 border p-6 rounded-md bg-base-100 shadow-2xl">
-            {showWelcomeRoll && <WelcomeRoll />}
+            <WelcomeRoll />
             <div className="flex justify-center mt-10">
               <ul className="menu menu-horizontal justify-center p-2 bg-base-300 rounded-full mb-8 w-fit mx-auto">
                 <li onClick={() => setGameState("joinGame")}>