Skip to content

Fix 404 logo links on THEOlive encoder overview pages - #826

Merged
MattiasBuelens merged 1 commit into
mainfrom
devin/1787314538-fix-theolive-encoder-links
Aug 21, 2026
Merged

Fix 404 logo links on THEOlive encoder overview pages#826
MattiasBuelens merged 1 commit into
mainfrom
devin/1787314538-fix-theolive-encoder-links

Conversation

@MattiasBuelens

Copy link
Copy Markdown
Collaborator

Summary

The logo cells in the THEOlive software/hardware encoder overview tables wrapped the image in a raw JSX anchor:

<a href="ffmpeg.md"><FfmpegLogo ... /></a>

Docusaurus only resolves Markdown links to doc files, so the literal ffmpeg.md leaked into the URL and resolved relative to the page, producing 404s like /docs/theolive/software-encoders/ffmpeg.md (reported by Algolia). The neighbouring Markdown links in the same rows (**[FFmpeg](ffmpeg.md)**) were fine.

Fix: turn each logo anchor into a Markdown link wrapping the JSX element, so it goes through the same link resolution:

[<FfmpegLogo ... />](ffmpeg.md)

All 8 logo links now point at the real doc URLs (/docs/theolive/contribution/software-encoders/ffmpeg/, /docs/theolive/hardware-encoders/haivision/, …) with the logo sizing unchanged. These two files were the only places in the docs using an internal doc path inside an <a href>; every other occurrence is an external URL, mailto:, or in-page anchor.

Link to Devin session: https://dolby.devinenterprise.com/sessions/ac9a43deeec54e17a3b07455be9dcc5f
Requested by: @MattiasBuelens

Co-Authored-By: Mattias Buelens <mattias.buelens@dolby.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@theoplayer-bot

This comment has been minimized.

@MattiasBuelens
MattiasBuelens merged commit d8d9e53 into main Aug 21, 2026
4 checks passed
@MattiasBuelens
MattiasBuelens deleted the devin/1787314538-fix-theolive-encoder-links branch August 21, 2026 12:51
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