From 83d98495b3d4ffe4958971720296b91a154ffd0f Mon Sep 17 00:00:00 2001
From: Kalle <38327916+Sendouc@users.noreply.github.com>
Date: Sat, 20 May 2023 22:34:23 +0300
Subject: [PATCH] Don't autorefresh brackets page if all matches are over

---
 .../tournament-bracket/routes/to.$id.brackets.tsx      | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/features/tournament-bracket/routes/to.$id.brackets.tsx b/app/features/tournament-bracket/routes/to.$id.brackets.tsx
index 1893ac059..1ebb92eb2 100644
--- a/app/features/tournament-bracket/routes/to.$id.brackets.tsx
+++ b/app/features/tournament-bracket/routes/to.$id.brackets.tsx
@@ -253,12 +253,11 @@ export default function TournamentBracketsPage() {
     };
   }, [data, navigate, parentRouteData]);
 
-  // TODO: also disable autorefresh (don't render component) and don't trigger revalidate after tournament is finalized
   React.useEffect(() => {
-    if (visibility !== "visible") return;
+    if (visibility !== "visible" || data.everyMatchIsOver) return;
 
     revalidate();
-  }, [visibility, revalidate]);
+  }, [visibility, revalidate, data.everyMatchIsOver]);
 
   const myTeam = parentRouteData.teams.find((team) =>
     team.members.some((m) => m.userId === user?.id)
@@ -266,7 +265,9 @@ export default function TournamentBracketsPage() {
 
   return (
     <div>
-      {visibility !== "hidden" ? <AutoRefresher /> : null}
+      {visibility !== "hidden" && !data.everyMatchIsOver ? (
+        <AutoRefresher />
+      ) : null}
       {!data.hasStarted && data.enoughTeams ? (
         <Form method="post" className="stack items-center">
           {!canAdminTournament({ user, event: parentRouteData.event }) ? (
@@ -309,7 +310,6 @@ export default function TournamentBracketsPage() {
   );
 }
 
-// TODO: don't render this guy if tournament is over
 function AutoRefresher() {
   useAutoRefresh();