Skip to content

fix: use production prefix for blog media - #829

Merged
up-the-tempo merged 1 commit into
mainfrom
centaur/fix-production-blog-media-prefix-1787158931
Aug 19, 2026
Merged

fix: use production prefix for blog media#829
up-the-tempo merged 1 commit into
mainfrom
centaur/fix-production-blog-media-prefix-1787158931

Conversation

@decofe

@decofe decofe commented Aug 19, 2026

Copy link
Copy Markdown
Member

Summary

  • Use explicit /developers/blog/... media URLs in production.
  • Keep /blog/... media URLs in previews and local development.
  • Cover both environments with regression tests.

Root cause

PR #828 converted media URLs to relative paths, but the marketing page has <base href="/">, so browsers resolved them from the domain root instead of the article directory.

Testing

  • pnpm exec biome check src/marketing/blogPlugin.ts src/marketing/blogPlugin.test.ts
  • pnpm check:types
  • Generated production blog data contains /developers/blog/... for every Voight-Kampff image and video
  • Verified the production /developers/blog/... assets return their correct PNG and MP4 content types

Prompted by: tom

@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
tempo-docs Ready Ready Preview Aug 19, 2026 5:07pm

Request Review

@up-the-tempo
up-the-tempo merged commit 0673cf2 into main Aug 19, 2026
10 of 11 checks passed
@up-the-tempo
up-the-tempo deleted the centaur/fix-production-blog-media-prefix-1787158931 branch August 19, 2026 17:08
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