Skip to content

fix: include night video URL in theme-config API response#91

Merged
fatherlinux merged 1 commit intomasterfrom
bugfix/night-theme-video
Feb 10, 2026
Merged

fix: include night video URL in theme-config API response#91
fatherlinux merged 1 commit intomasterfrom
bugfix/night-theme-video

Conversation

@fatherlinux
Copy link
Member

Summary

  • Fixed bug where night theme video wasn't loading because /api/theme-config didn't include the night video URL in the video_urls response
  • The code only iterated through config.themes[] (7 seasonal themes) but didn't check config.nightMode for the night video

Test plan

  • Verified curl http://localhost:8080/api/theme-config | jq '.video_urls' now includes "night": "/api/theme-video/night"
  • Verified /api/theme-video/night returns 200 OK with video/mp4 content
  • All tests pass (Vitest, Gourmand, ESLint)

🤖 Generated with Claude Code

The /api/theme-config endpoint was only iterating through config.themes[]
(the 7 seasonal themes) but not adding the night video URL. Night mode is
configured separately in config.nightMode, so its video URL was never
generated.

Added code to check if nightMode is enabled and include the night video
URL in the video_urls response.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@fatherlinux fatherlinux merged commit efd1f3c into master Feb 10, 2026
2 checks passed
@fatherlinux fatherlinux deleted the bugfix/night-theme-video branch February 10, 2026 16:26
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.

1 participant