Skip to content

fix(research): render the Q&A podcast from the CDN only - #62

Closed
jfrench9 wants to merge 1 commit into
mainfrom
bugfix/research-podcast-cdn-only
Closed

jfrench9 wants to merge 1 commit into
mainfrom
bugfix/research-podcast-cdn-only

Conversation

@jfrench9

@jfrench9 jfrench9 commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

The Q&A podcast uploads were removed from YouTube, and ResearchArticle still fell back to a YouTube embed of podcast_youtube_url when no CDN MP3 existed. Ticker pages (GTBIF and the 17 others carrying that field) rendered a "Listen" section with YouTube's "Video unavailable" card.

  • The podcast section now renders only when assets.podcast_mp3 exists, in the native audio player from the CDN.
  • The YouTube embed fallback and the "Watch on YouTube" link are gone; podcast_youtube_url is kept on the type as catalog data and annotated as never rendered.
  • The report video itself is unchanged and stays on YouTube.
  • Test covers all three cases: YouTube-only podcast renders nothing, CDN MP3 renders the player with no YouTube reference, the report video embed is untouched.

Today no catalog entry carries a podcast_mp3, so the section disappears everywhere until a podcast is published to the CDN. The content-machine reindex maps assets by filename suffix (_final.mp4, _short.mp4, _brief.md, _thumbnail.png) and has no podcast suffix, so bringing the podcast back is a reindex change as well as an upload.

Rollout

Release core, then bump @robosystems/core in roboinvestor-app (the only app rendering research now).

🤖 Generated with Claude Code

https://claude.ai/code/session_01BNVctiLqwswiMoiDFgwXku

The podcast YouTube uploads were removed, so ticker pages showed YouTube's
"Video unavailable" card under "Listen". The podcast section now renders only
when the catalog carries a CDN MP3, in the native player, and never embeds or
links the catalog's podcast_youtube_url. The report video stays on YouTube.

Claude-Session: https://claude.ai/code/session_01BNVctiLqwswiMoiDFgwXku
@jfrench9

jfrench9 commented Sep 3, 2026

Copy link
Copy Markdown
Member Author

Superseded: the research module moves into roboinvestor-app (its only consumer) with this fix applied there; core gets a removal PR instead.

@jfrench9 jfrench9 closed this Sep 3, 2026
@jfrench9
jfrench9 deleted the bugfix/research-podcast-cdn-only branch September 9, 2026 17:16
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