Skip to content

Fix asset generation for news episodes with title-based templates#138

Merged
denbec merged 1 commit intomainfrom
denbec/fix-news-cover-asset
Feb 11, 2026
Merged

Fix asset generation for news episodes with title-based templates#138
denbec merged 1 commit intomainfrom
denbec/fix-news-cover-asset

Conversation

@denbec
Copy link
Collaborator

@denbec denbec commented Feb 11, 2026

Summary

When multiple asset templates match the same asset_type with different title filters (e.g., "AI News" vs regular "News"), only the most specific template now runs. This prevents generating both variants and setting the wrong cover image.

Also fixed a stale state bug where cover_image and banner_image weren't updated in memory, causing fallback templates to overwrite the initially-set cover image.

Changes

  • Added priority-based template filtering: generic templates are excluded when a specific template (with title_contains) matches for the same asset_type
  • Update podcast object state after DB updates to ensure subsequent templates see the current state

Co-Authored-By: Claude Haiku 4.5 noreply@anthropic.com

…ction

When multiple templates match the same asset_type with different title filters
(e.g., "AI News" vs regular "News"), only the most specific template should run.
Also update podcast's cover_image and banner_image state after DB update to
prevent stale checks from overwriting with a fallback template.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Feb 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
programmierbar-website Ready Ready Preview, Comment Feb 11, 2026 0:41am

Request Review

@denbec denbec merged commit 0300cc0 into main Feb 11, 2026
3 checks passed
@denbec denbec deleted the denbec/fix-news-cover-asset branch February 13, 2026 12:41
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