Skip to content

docs: fix MLflow video embed#16496

Merged
gr2m merged 1 commit into
mainfrom
codex/fix-mlflow-video
Jun 29, 2026
Merged

docs: fix MLflow video embed#16496
gr2m merged 1 commit into
mainfrom
codex/fix-mlflow-video

Conversation

@gr2m

@gr2m gr2m commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Background

The MLflow observability page used Markdown image syntax for an .mp4 asset:

![MLflow Tracing Vercel AI SDK](https://mlflow.org/docs/latest/images/llms/tracing/vercel-ai-tracing.mp4)

That produces an image element, so the browser renders the hero media as a broken image even though the asset itself returns 200 OK with content-type: video/mp4.

I compared the two open PRs for #12472:

Summary

This updates the current MLflow observability page at content/providers/03-observability/mlflow.mdx to render the MP4 with a <video> element. The video keeps controls, uses muted for reliable autoplay, loops like a hero demo, and includes a fallback link inside the element.

Checklist

  • All commits are signed (PRs with unsigned commits cannot be merged)
  • Tests have been added / updated (for bug fixes / features)
  • Documentation has been added / updated (for bug fixes / features)
  • A patch changeset for relevant packages has been added (for bug fixes / features - run pnpm changeset in the project root)
  • I have reviewed this pull request (self-review)

Related Issues

Fixes #12472.
Supersedes and closes #12677.
Supersedes and closes #15924.

Co-authored-by: Shurik shurik@openclaw.ai
Co-authored-by: Narutama Aurum narutamaaurum@gmail.com

Co-authored-by: Shurik <shurik@openclaw.ai>

Co-authored-by: Narutama Aurum <narutamaaurum@gmail.com>
@gr2m gr2m force-pushed the codex/fix-mlflow-video branch from 1ee354b to ad87d7e Compare June 29, 2026 22:48
@gr2m gr2m added the documentation Improvements or additions to documentation label Jun 29, 2026
@gr2m gr2m merged commit a131e78 into main Jun 29, 2026
48 checks passed
@gr2m gr2m deleted the codex/fix-mlflow-video branch June 29, 2026 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Video in documentation is broken

1 participant