Skip to content

docs: remove self-hosted video embeds from all doc pages#1326

Merged
rohitg00 merged 1 commit intomainfrom
docs/remove-page-videos
Mar 18, 2026
Merged

docs: remove self-hosted video embeds from all doc pages#1326
rohitg00 merged 1 commit intomainfrom
docs/remove-page-videos

Conversation

@rohitg00
Copy link
Contributor

@rohitg00 rohitg00 commented Mar 18, 2026

This pull request removes embedded video elements from several documentation pages in the Motia framework. The videos previously provided visual demonstrations for key concepts but have now been omitted, possibly to streamline the docs or improve load times.

Documentation updates:

  • Removed the video demonstration from the Why Motia? introduction page (frameworks/motia/docs/content/docs/why-motia.mdx).
  • Removed the video illustrating the "Steps" primitive from the steps concept page (frameworks/motia/docs/content/docs/concepts/steps.mdx).
  • Removed the video on observability features from the observability guide (frameworks/motia/docs/content/docs/development-guide/observability.mdx).
  • Removed the video explaining state management from the state management guide (frameworks/motia/docs/content/docs/development-guide/state-management.mdx).
  • Removed the video about real-time streams from the streams guide (frameworks/motia/docs/content/docs/development-guide/streams.mdx).

Summary by CodeRabbit

  • Documentation
    • Removed embedded video demonstrations from documentation pages covering Steps, Observability, State Management, Real-time Streams, and Why-Motia concepts. All written content and guides remain intact and accessible.

@vercel
Copy link
Contributor

vercel bot commented Mar 18, 2026

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

Project Deployment Actions Updated (UTC)
iii-website Ready Ready Preview, Comment Mar 18, 2026 11:24am
motia-docs Ready Ready Preview, Comment Mar 18, 2026 11:24am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 18, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: fe893f02-09b8-47c8-bc75-b38eeffb04e5

📥 Commits

Reviewing files that changed from the base of the PR and between 115674a and 197bd09.

📒 Files selected for processing (5)
  • frameworks/motia/docs/content/docs/concepts/steps.mdx
  • frameworks/motia/docs/content/docs/development-guide/observability.mdx
  • frameworks/motia/docs/content/docs/development-guide/state-management.mdx
  • frameworks/motia/docs/content/docs/development-guide/streams.mdx
  • frameworks/motia/docs/content/docs/why-motia.mdx
💤 Files with no reviewable changes (5)
  • frameworks/motia/docs/content/docs/concepts/steps.mdx
  • frameworks/motia/docs/content/docs/development-guide/state-management.mdx
  • frameworks/motia/docs/content/docs/development-guide/streams.mdx
  • frameworks/motia/docs/content/docs/development-guide/observability.mdx
  • frameworks/motia/docs/content/docs/why-motia.mdx

📝 Walkthrough

Walkthrough

Five MDX documentation files have had embedded video blocks removed from their content. These are presentation-only changes affecting visual media without any functional, structural, or API-level modifications to the documentation system.

Changes

Cohort / File(s) Summary
Video Embed Removal from Documentation
frameworks/motia/docs/content/docs/concepts/steps.mdx, frameworks/motia/docs/content/docs/development-guide/observability.mdx, frameworks/motia/docs/content/docs/development-guide/state-management.mdx, frameworks/motia/docs/content/docs/development-guide/streams.mdx, frameworks/motia/docs/content/docs/why-motia.mdx
Removed embedded video blocks (poster and source elements) from five documentation pages; pure content removal with no functional impact.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Videos hop away, the docs stay clear,
Five pages lighter, nothing to fear!
No logic harmed, just media gone,
Content still strong, the docs live on! 🎬✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: removing self-hosted video embeds from documentation pages, which matches all five files modified.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/remove-page-videos
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rohitg00 rohitg00 requested a review from sergiofilhowz March 18, 2026 11:26
@rohitg00 rohitg00 merged commit a90a4e6 into main Mar 18, 2026
22 checks passed
@rohitg00 rohitg00 deleted the docs/remove-page-videos branch March 18, 2026 15:10
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