From 80973d321206888de4370b113edf09d3a5ad93ea Mon Sep 17 00:00:00 2001 From: Chris Parsons <438676+chrisparsons83@users.noreply.github.com> Date: Wed, 20 May 2026 00:02:19 -0700 Subject: [PATCH] Fix draft Summary tab showing wrong avatar for teams without custom logos (#453) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit DraftSummaryView was not declaring or forwarding ownerAvatarData, so TeamAvatar fell back to a generated flag instead of the owner's avatar. DraftGrid and DraftGridSection already threaded ownerAvatarData correctly; this makes the Summary tab consistent. No route changes needed — the loader already enriches draftSlots with ownerAvatarData. Co-authored-by: Claude Sonnet 4.6 --- app/components/draft/DraftSummaryView.tsx | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/components/draft/DraftSummaryView.tsx b/app/components/draft/DraftSummaryView.tsx index afb75ae..650285f 100644 --- a/app/components/draft/DraftSummaryView.tsx +++ b/app/components/draft/DraftSummaryView.tsx @@ -1,7 +1,7 @@ import { memo, useMemo, Fragment } from "react"; import { TeamAvatar } from "~/components/TeamAvatar"; import { type DraftPick, groupPicksByTeamAndSport } from "./picks-utils"; -import type { RawFlagConfig } from "~/lib/flag-types"; +import type { AvatarData, RawFlagConfig } from "~/lib/flag-types"; interface DraftSummaryViewProps { draftSlots: Array<{ @@ -12,6 +12,7 @@ interface DraftSummaryViewProps { logoUrl?: string | null; flagConfig?: RawFlagConfig | null; avatarType?: string | null; + ownerAvatarData?: AvatarData | null; }; }>; ownerMap?: Record; @@ -102,6 +103,7 @@ export const DraftSummaryView = memo(function DraftSummaryView({ logoUrl={slot.team.logoUrl} flagConfig={slot.team.flagConfig} avatarType={slot.team.avatarType} + ownerAvatarData={slot.team.ownerAvatarData} size="md" />