ChatCodes revisited

app/features/chat/useChatContext.ts

@@ -1,6 +1,8 @@
 import * as React from "react";
 import type { ChatContextValue } from "./chat-provider-types";
 
+// xxx: think how chats should expire in relation to chatCode getting returned from loaders etc.
+
 export const ChatContext = React.createContext<ChatContextValue | null>(null);
 
 export function useChatContext(): ChatContextValue | null {

app/features/scrims/loaders/scrims.$id.server.ts

@@ -29,7 +29,7 @@ export const loader = async ({ params }: LoaderFunctionArgs) => {
 	return {
 		post,
 		chatCode:
-			user.roles.includes("STAFF") && !participantIds.includes(user.id)
+			user.roles.includes("STAFF") || participantIds.includes(user.id)
 				? post.chatCode
 				: undefined,
 		anyUserPrefersNoScreen:

app/features/sendouq-match/loaders/q.match.$id.server.ts

@@ -43,9 +43,8 @@ export const loader = async ({ params }: LoaderFunctionArgs) => {
 				})
 			: null,
 		rawReportedWeapons,
-		// xxx: as a safety measure, it'd be good to send chat code also when user is in the match
 		chatCode:
-			user?.roles.includes("STAFF") && !matchUsers.includes(user.id)
+			user?.roles.includes("STAFF") || (user && matchUsers.includes(user.id))
 				? match.chatCode
 				: null,
 	};

app/features/sendouq/loaders/q.looking.server.ts

@@ -46,5 +46,6 @@ export const loader = async ({ request }: LoaderFunctionArgs) => {
 				},
 		lastUpdated: Date.now(),
 		streamsCount: (await cachedStreams()).length,
+		chatCode: ownGroup ? ownGroup.chatCode : null,
 	};
 };

app/features/tournament-bracket/loaders/to.$id.matches.$mid.server.ts

@@ -123,8 +123,8 @@ export const loader = async ({ params }: LoaderFunctionArgs) => {
 		endedEarly,
 		noScreen,
 		chatCode:
-			tournament.isOrganizerOrStreamer(user) &&
-			!match.players.some((p) => p.id === user?.id)
+			tournament.isOrganizerOrStreamer(user) ||
+			match.players.some((p) => p.id === user?.id)
 				? match.chatCode
 				: undefined,
 	};