Merge pull request #98 from cardano-scaling/feat/force-login-join

Force login to join game
cardano-scaling · Dec 14, 2024 · 3e32fb7 · 3e32fb7
2 parents e6eaed8 + 61585a9
commit 3e32fb7
Show file tree

Hide file tree

Showing 4 changed files with 39 additions and 14 deletions.
diff --git a/src/components/InitialView/InitialView.tsx b/src/components/InitialView/InitialView.tsx
@@ -17,20 +17,22 @@ interface InitialViewProps {
 
 const InitialView: FC<InitialViewProps> = ({ startGame }) => {
   const {
-    setGameData,
     accountData,
-    isQualified,
-    setIsQualified,
     isLoadingUserData,
+    isQualified,
+    isUserDataFetched,
+    keys,
     setAccountData,
+    setGameData,
+    setIsQualified,
   } = useAppContext();
+const publicKeyHashHex = keys?.publicKeyHashHex
   const [, setSessionId] = useSessionIdKeyCache();
   const pathSegments = window.location.hash.split("/").filter(Boolean);
   const code = pathSegments[2];
   const [modalTitle, _] = useState("Join Multiplayer");
-  const [isNameModalOpen, setIsNameModalOpen] = useState(
-    pathSegments[1] === EGameType.JOIN,
-  );
+  const [isNameModalOpen, setIsNameModalOpen] = useState(false);
+  const isJoin = pathSegments[1] === EGameType.JOIN;
 
   const [isWelcomeModalOpen, setIsWelcomeModalOpen] = useState(false);
   const [isLoginModalOpen, setIsLoginModalOpen] = useState(false);
@@ -42,6 +44,16 @@ const InitialView: FC<InitialViewProps> = ({ startGame }) => {
     }
   }, [code, setGameData]);
 
+  useEffect(() => {
+    if (isJoin && publicKeyHashHex && !isLoadingUserData) {
+      if (isUserDataFetched) {
+        setIsNameModalOpen(true);
+      } else {
+        setIsLoginModalOpen(true);
+      }
+    }
+  }, [isLoadingUserData, isUserDataFetched, isJoin, publicKeyHashHex]);
+
   const handleTournamentLogin = () => {
     setIsLoginModalOpen(true);
   };
@@ -188,7 +200,9 @@ const InitialView: FC<InitialViewProps> = ({ startGame }) => {
       />
       <LoginModal
         close={() => setIsLoginModalOpen(false)}
+        isJoin={isJoin}
         isOpen={isLoginModalOpen}
+        openNameModal={() => setIsNameModalOpen(true)}
         showActionButtons={showActionButtons}
       />
     </Layout>

diff --git a/src/components/LoginModal/LoginModal.tsx b/src/components/LoginModal/LoginModal.tsx
@@ -26,7 +26,9 @@ import { checkSignin, fetchAuthProviders } from "../../utils/requests";
 
 interface LoginModalProps {
   close: () => void;
+  isJoin: boolean;
   isOpen: boolean;
+  openNameModal: () => void;
   showActionButtons: () => void;
 }
 
@@ -77,7 +79,9 @@ const CheckBoxInput: FC<{
 
 const LoginModal: React.FC<LoginModalProps> = ({
   close,
+  isJoin,
   isOpen,
+  openNameModal,
   showActionButtons,
 }) => {
   const shouldShowAllTou = Date.now() >= 1733238000000;
@@ -120,14 +124,17 @@ const LoginModal: React.FC<LoginModalProps> = ({
       if (account) setAccountData(account);
       if (session?.session_id) setSessionId(session.session_id);
       if (qualifier) setIsQualified(qualifier.is_qualified);
+      if (isJoin) openNameModal();
     }
   }, [
     close,
+    isJoin,
+    openNameModal,
     setAccountData,
+    setIsQualified,
     setSessionId,
     showActionButtons,
     userData,
-    setIsQualified,
   ]);
 
   const handleClose = () => {
@@ -154,7 +161,7 @@ const LoginModal: React.FC<LoginModalProps> = ({
   const renderConsentContent = () => {
     return (
       <div className="text-left flex flex-col gap-4">
-        <h1 className="text-5xl uppercase">Tournament Consent</h1>
+        <h1 className="text-5xl uppercase mb-6">Tournament Consent</h1>
         {shouldShowAllTou ? (
           <>
             {/**

diff --git a/src/context/AppContextProvider.tsx b/src/context/AppContextProvider.tsx
@@ -82,18 +82,19 @@ const AppContextProvider: FC<PropsWithChildren> = ({ children }) => {
       bots,
       gameData,
       globalStats,
+      isUserDataFetched: !!(userData || accountData),
       isLoadingGlobalStats,
       isLoadingUserData,
+      isQualified,
       keys,
       players,
       region,
       setAccountData,
       setBots,
       setGameData,
+      setIsQualified,
       setPlayers,
       setRegion,
-      isQualified,
-      setIsQualified,
     }),
     [
       accountData,
@@ -107,6 +108,7 @@ const AppContextProvider: FC<PropsWithChildren> = ({ children }) => {
       players,
       region,
       isQualified,
+      userData,
     ],
   );
 

diff --git a/src/context/useAppContext.ts b/src/context/useAppContext.ts
@@ -14,18 +14,19 @@ interface AppContextInterface {
   bots: number;
   gameData: GameData;
   globalStats?: GameStatistics;
+  isUserDataFetched: boolean;
   isLoadingGlobalStats: boolean;
   isLoadingUserData: boolean;
+  isQualified: boolean;
   keys: Keys | null;
   players: number;
   region: Region | null;
-  isQualified: boolean;
   setAccountData: Dispatch<React.SetStateAction<Account | undefined>>;
   setBots: Dispatch<React.SetStateAction<number>>;
   setGameData: Dispatch<React.SetStateAction<GameData>>;
+  setIsQualified: Dispatch<React.SetStateAction<boolean>>;
   setPlayers: Dispatch<React.SetStateAction<number>>;
   setRegion: Dispatch<React.SetStateAction<Region | null>>;
-  setIsQualified: Dispatch<React.SetStateAction<boolean>>;
 }
 
 export const AppContext = createContext<AppContextInterface>({
@@ -34,18 +35,19 @@ export const AppContext = createContext<AppContextInterface>({
   bots: 1,
   gameData: { petName: "", code: "", type: EGameType.SOLO },
   globalStats: undefined,
+  isUserDataFetched: false,
   isLoadingGlobalStats: false,
   isLoadingUserData: false,
+  isQualified: false,
   keys: null,
   players: 1,
   region: null,
-  isQualified: false,
   setAccountData: () => {},
   setBots: () => {},
   setGameData: () => {},
+  setIsQualified: () => {},
   setPlayers: () => {},
   setRegion: () => {},
-  setIsQualified: () => {},
 });
 
 export const useAppContext = () => {