Skip to content

feat: show season names in season selector dropdown - #194

Merged
dannyvfilms merged 1 commit into
dannyvfilms:latestfrom
nakenyon:feat/season-names-in-dropdown
May 14, 2026
Merged

feat: show season names in season selector dropdown#194
dannyvfilms merged 1 commit into
dannyvfilms:latestfrom
nakenyon:feat/season-names-in-dropdown

Conversation

@nakenyon

@nakenyon nakenyon commented May 6, 2026

Copy link
Copy Markdown

Summary

  • Season dropdown on TV show detail pages now shows the actual season name (e.g. "Fantasy High", "Never Stop Blowing Up") instead of the show title
  • Falls back to "Season X" / "Specials" for shows where TMDB/TVDB only provides generic season names, so existing behaviour is unchanged for those
  • Root cause: season_title (populated by the provider) was being ignored in favour of season_header_title, which is only set on the local-only data path and caused a 500 when used as a Django filter argument on the normal path

Changes

  • src/templates/app/media_details.html — use season_title in the dropdown item render instead of season_header_title
  • src/app/views.py — also prefer provider season_title over generic fallback when building season_header_title for local-only season entries

Test plan

  • Open a TV show with named seasons (e.g. Dimension 20) — dropdown should show real season names
  • Open a TV show with generic seasons (e.g. Breaking Bad) — dropdown should still show "Season 1", "Season 2", etc.
  • Verify no 500 errors on season detail pages

🤖 Generated with Claude Code

Display the actual season name (e.g. "Fantasy High", "Never Stop Blowing
Up") in the season dropdown instead of the show title. Falls back to
"Season X" for shows with generic season names, so existing behaviour
is unchanged for those.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@dannyvfilms
dannyvfilms merged commit e703229 into dannyvfilms:latest May 14, 2026
5 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants