From bc23b7c75cda6a2e6ac43042cbf6c50b11886fc3 Mon Sep 17 00:00:00 2001 From: Chuck <33324927+ChuckBuilds@users.noreply.github.com> Date: Mon, 19 Jan 2026 09:48:52 -0500 Subject: [PATCH] fix(logos): Add ncaam/ncaaw sport key aliases for basketball plugin (#200) The basketball scoreboard plugin uses sport_key="ncaam" and "ncaaw", but LogoDownloader only had "ncaam_basketball" mapped. This caused get_logo_directory() to fall back to "assets/sports/ncaam_logos" (non-existent) instead of "assets/sports/ncaa_logos". Added aliases to both LOGO_DIRECTORIES and API_ENDPOINTS: - ncaam -> assets/sports/ncaa_logos - ncaaw -> assets/sports/ncaa_logos - ncaaw_basketball -> assets/sports/ncaa_logos Co-authored-by: Chuck Co-authored-by: Claude Opus 4.5 --- src/logo_downloader.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/logo_downloader.py b/src/logo_downloader.py index 495f5c7c..5fdb76e0 100644 --- a/src/logo_downloader.py +++ b/src/logo_downloader.py @@ -37,6 +37,9 @@ class LogoDownloader: 'ncaa_fb_all': 'https://site.api.espn.com/apis/site/v2/sports/football/college-football/teams', # Includes FCS 'fcs': 'https://site.api.espn.com/apis/site/v2/sports/football/college-football/teams', # FCS teams from same endpoint 'ncaam_basketball': 'https://site.api.espn.com/apis/site/v2/sports/basketball/mens-college-basketball/teams', + 'ncaam': 'https://site.api.espn.com/apis/site/v2/sports/basketball/mens-college-basketball/teams', # Alias for basketball plugin + 'ncaaw_basketball': 'https://site.api.espn.com/apis/site/v2/sports/basketball/womens-college-basketball/teams', + 'ncaaw': 'https://site.api.espn.com/apis/site/v2/sports/basketball/womens-college-basketball/teams', # Alias for basketball plugin 'ncaa_baseball': 'https://site.api.espn.com/apis/site/v2/sports/baseball/college-baseball/teams', 'ncaam_hockey': 'https://site.api.espn.com/apis/site/v2/sports/hockey/mens-college-hockey/teams', # Soccer leagues @@ -63,6 +66,9 @@ class LogoDownloader: 'ncaa_fb_all': 'assets/sports/ncaa_logos', 'fcs': 'assets/sports/ncaa_logos', 'ncaam_basketball': 'assets/sports/ncaa_logos', + 'ncaam': 'assets/sports/ncaa_logos', # Alias for basketball plugin + 'ncaaw_basketball': 'assets/sports/ncaa_logos', + 'ncaaw': 'assets/sports/ncaa_logos', # Alias for basketball plugin 'ncaa_baseball': 'assets/sports/ncaa_logos', 'ncaam_hockey': 'assets/sports/ncaa_logos', 'ncaaw_hockey': 'assets/sports/ncaa_logos',