feat(podcast): YouTube rendering + Andrew Pla & Jordan Hammond profiles#36
Merged
Merged
Conversation
✅ Deploy Preview for powershellorg ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
List cards and episode pages hotlink the YouTube thumbnail (maxresdefault, onerror fallback to hqdefault) when an episode has a `youtube` id, falling back to the existing purple icon otherwise. The single page adds a click-to-load embed facade (thumbnail poster -> youtube-nocookie iframe on click) alongside the existing audio player, avoiding eager iframes across 220 episodes. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Scaffold author profiles so the rewritten podcast bylines resolve to real bio cards instead of bare taxonomy pages (per ADR 0002). Slugs match Hugo's slug of the exact `authors:` strings. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The click-to-load facade relied on Tailwind's `aspect-video` utility, which does not exist in the theme's Tailwind v2 CDN build — once the poster image was swapped for the iframe the container collapsed to zero height. Set the aspect ratio via the CSS `aspect-ratio` property instead. Also apply the YouTube-thumbnail-with-icon-fallback to the episode cards on author profile pages, which were always rendering the podcast icon. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Replace the generic fa-user icon next to the byline with the author's avatar, resolving the byline to its taxonomy term page and reusing the author-avatar partial (avatar -> gravatar -> identicon fallback), so un-enriched authors still get a sensible image. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2273735 to
79fbfd0
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Implements WS5 (rendering) and WS6 (author profiles) from
docs/podcast-sync-plan.md. Both are small and presentation-only, so they share one PR.WS5 — YouTube rendering
podcast/list.html: cards hotlinkimg.youtube.com/vi/<id>/maxresdefault.jpg(withonerrorfallback tohqdefault.jpg) when an episode has ayoutubeid; otherwise the existing purple podcast icon._default/single.html: adds a click-to-load YouTube embed facade (thumbnail poster →youtube-nocookie.comiframe on click; keyboard-accessible) alongside the existing audio player, gated onyoutube. Avoids eager iframes across the 220 episodes.youtube(incl. archived PowerScripting/libsyn episodes).WS6 — Author profiles
content/authors/andrew-pla/andcontent/authors/jordan-hammond/viatools/new-author.ps1so slugs match the rewritten bylines (per ADR 0002) — no orphan pages.Verification
hugo --gc --minifybuilds clean; no orphan-page warnings./authors/andrew-pla/and/authors/jordan-hammond/render card + bio and list their episodes; episode "About the Author" now resolves to Andrew.🤖 Generated with Claude Code