From 5cbe1ea482e03a0ee1904e5c59e25fc29df79c8e Mon Sep 17 00:00:00 2001
From: Kalle <38327916+Sendouc@users.noreply.github.com>
Date: Sat, 16 Mar 2024 12:01:47 +0200
Subject: [PATCH] Add subs popover with info text visible to team member Closes
 #1689

---
 .../routes/to.$id.brackets.tsx                  | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/app/features/tournament-bracket/routes/to.$id.brackets.tsx b/app/features/tournament-bracket/routes/to.$id.brackets.tsx
index fdb0a7b89..0f20ebb82 100644
--- a/app/features/tournament-bracket/routes/to.$id.brackets.tsx
+++ b/app/features/tournament-bracket/routes/to.$id.brackets.tsx
@@ -237,7 +237,6 @@ export default function TournamentBracketsPage() {
   const showAddSubsButton =
     !tournament.canFinalize(user) &&
     !tournament.everyBracketOver &&
-    tournament.ownedTeamByUser(user) &&
     tournament.hasStarted;
 
   const waitingForTeamsText = () => {
@@ -465,7 +464,21 @@ function AddSubsPopOver() {
   const user = useUser();
 
   const ownedTeam = tournament.ownedTeamByUser(user);
-  invariant(ownedTeam, "User doesn't have owned team");
+  if (!ownedTeam) {
+    const teamMemberOf = tournament.teamMemberOfByUser(user);
+    if (!teamMemberOf) return null;
+
+    return (
+      <Popover
+        buttonChildren={<>{t("tournament:actions.addSub")}</>}
+        triggerClassName="tiny outlined ml-auto"
+        triggerTestId="add-sub-button"
+        contentClassName="text-xs"
+      >
+        Only team captain or a TO can add subs
+      </Popover>
+    );
+  }
 
   const subsAvailableToAdd =
     tournament.maxTeamMemberCount - ownedTeam.members.length;