Linting and Formatting

TheBozzz34 · Mar 8, 2024 · 1516f3c · 1516f3c
1 parent 2c57cde
commit 1516f3c
Show file tree

Hide file tree

Showing 2 changed files with 76 additions and 67 deletions.
diff --git a/pages/app/index.tsx b/pages/app/index.tsx
@@ -1,4 +1,4 @@
-import { User, onAuthStateChanged } from "firebase/auth"
+import { onAuthStateChanged, User } from "firebase/auth"
 import Head from "next/head"
 import { useRouter } from "next/navigation"
 import React, { useEffect, useRef, useState } from "react"
@@ -10,33 +10,35 @@ import { auth } from "../../firebase"
 import { useIsMobile } from "../../utils/useIsMobile"
 
 export default function Scoreboard() {
-  const [userAccount, setUserAccount] = useState<User | null>(null);
-
-    useEffect(() => {
-        onAuthStateChanged(auth, (user) => {
-            if (user) {
-                setUserAccount(user);
-            } else {
-                setUserAccount(null);
-            }
-        });
-    }, []);
-
-    let ignore = false;
-
-    useEffect(() => {
-
-      if (ignore) return;
-
-      const message = {
-        action: 'get',
-        boardId: userAccount?.uid
+  const [userAccount, setUserAccount] = useState<User | null>(null)
+  const { messages, sendMessage, connectionStatus } = useWebSocket()
+
+  const ignore = useRef<boolean>(false)
+
+  useEffect(() => {
+    onAuthStateChanged(auth, (user) => {
+      if (user) {
+        setUserAccount(user)
+      } else {
+        setUserAccount(null)
       }
-      sendMessage(JSON.stringify(message))
-      ignore = true;
-    }, [userAccount])
+    })
+  }, [])
+
+  useEffect(() => {
+    if (ignore.current) return
+
+    const message = {
+      action: "get",
+      boardId: userAccount?.uid,
+    }
+    sendMessage(JSON.stringify(message))
+
+    return () => {
+      ignore.current = true
+    }
+  }, [sendMessage, userAccount])
 
-  const { messages, sendMessage, connectionStatus } = useWebSocket()
   const [teamAScore, setTeamAScore] = useState(0)
   const [teamBScore, setTeamBScore] = useState(0)
   const [teamAFouls, setTeamAFouls] = useState(0)
@@ -117,7 +119,6 @@ export default function Scoreboard() {
 
   useEffect(() => {
     console.log("Updating board")
-
   }, [
     teamAName,
     teamBName,
@@ -128,12 +129,12 @@ export default function Scoreboard() {
     currentPeriod,
     minutes,
     seconds,
-    possession
+    possession,
   ])
 
   useEffect(() => {
     messages.forEach((message) => {
-      const messageJson = JSON.parse(message) as { boardId: string, boardData: string }
+      const messageJson = JSON.parse(message) as { boardId: string; boardData: string }
       if (messageJson.boardId === userAccount?.uid) {
         const data = JSON.parse(messageJson.boardData)
         setTeamAName((data as { teamAName: string }).teamAName)
@@ -148,20 +149,31 @@ export default function Scoreboard() {
         router.push("/login")
       }
     })
-  }, [router, sendMessage, messages])
+  }, [router, sendMessage, messages, userAccount?.uid])
 
   useEffect(() => {
     reportData()
-  }, [teamAName, teamBName, teamAScore, teamBScore, teamAFouls, teamBFouls, currentPeriod, minutes, seconds, possession])
-
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [
+    teamAName,
+    teamBName,
+    teamAScore,
+    teamBScore,
+    teamAFouls,
+    teamBFouls,
+    currentPeriod,
+    minutes,
+    seconds,
+    possession,
+  ])
 
   const incrementTeamAScore = () => {
-    setTeamAScore(teamAScore + 1) 
+    setTeamAScore(teamAScore + 1)
   }
 
   // Function to increment the score for Team B
   const incrementTeamBScore = () => {
-    setTeamBScore(teamBScore + 1) 
+    setTeamBScore(teamBScore + 1)
   }
 
   const handleTeamANameChange = (newName: React.SetStateAction<string>) => {

diff --git a/pages/public/index.tsx b/pages/public/index.tsx
@@ -1,49 +1,46 @@
-import { useWebSocket } from "context/WebSocketContext"
-import { auth } from "../../firebase"
-import { User, onAuthStateChanged } from "firebase/auth"
+import { onAuthStateChanged, User } from "firebase/auth"
 import Head from "next/head"
-import React, { useEffect, useState } from "react"
 import { useRouter } from "next/router"
+import React, { useEffect, useState } from "react"
+import { useWebSocket } from "context/WebSocketContext"
+import { auth } from "../../firebase"
 
 const Public = () => {
-    const { messages, sendMessage, connectionStatus } = useWebSocket()
-    const [userAccount, setUserAccount] = useState<User | null>(null);
-    const router = useRouter();
-
-    useEffect(() => {
-        onAuthStateChanged(auth, (user) => {
-            if (user) {
-                setUserAccount(user);
-            } else {
-                setUserAccount(null);
-            }
-        });
-    }, []);
+  const { messages, sendMessage } = useWebSocket()
+  const [userAccount, setUserAccount] = useState<User | null>(null)
+  const router = useRouter()
 
-    useEffect(() => {
-        messages.forEach((message) => {
-          const messageJson = JSON.parse(message) as { boardId: string, boardData: string }
-          if (messageJson.boardId === userAccount?.uid) {
-            const data = JSON.parse(messageJson.boardData)
-            console.log(data)
-          }
-        })
-
-        onAuthStateChanged(auth, (user) => {
-          if (!user) {
-            router.push("/login")
-          }
-        })
-      }, [router, sendMessage, messages])
+  useEffect(() => {
+    onAuthStateChanged(auth, (user) => {
+      if (user) {
+        setUserAccount(user)
+      } else {
+        setUserAccount(null)
+      }
+    })
+  }, [])
 
+  useEffect(() => {
+    messages.forEach((message) => {
+      const messageJson = JSON.parse(message) as { boardId: string; boardData: string }
+      if (messageJson.boardId === userAccount?.uid) {
+        const data = JSON.parse(messageJson.boardData)
+        console.log(data)
+      }
+    })
 
+    onAuthStateChanged(auth, (user) => {
+      if (!user) {
+        router.push("/login")
+      }
+    })
+  }, [router, sendMessage, messages, userAccount?.uid])
 
   return (
     <>
       <Head>
         <title>ScoreConnect Public Page</title>
       </Head>
-
     </>
   )
 }