Skip to content

fix(marketing): update video section to new YouTube video - #973

Merged
Kitenite merged 1 commit into
mainfrom
kiet-ho/reading
Jan 26, 2026
Merged

Kitenite merged 1 commit into
mainfrom
kiet-ho/reading

Conversation

@Kitenite

@Kitenite Kitenite commented Jan 26, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Update the marketing site video section with the new YouTube video (7jhPfMDwTUc)

Test plan

  • Verify the video plays correctly on the marketing site
  • Check that the thumbnail click triggers the video embed

Summary by CodeRabbit

  • Chores
    • Updated video content displayed in the marketing section.

✏️ Tip: You can customize this high-level summary in your review settings.

Replace the demo video with updated content (7jhPfMDwTUc)
@coderabbitai

coderabbitai Bot commented Jan 26, 2026 •

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

The VIDEO_ID constant in the VideoSection component is updated from "dkD-U7JXkbI" to "7jhPfMDwTUc", changing which YouTube video is embedded and displayed when the video player is active.

Changes

Cohort / File(s) Summary
Video Component Update
apps/marketing/src/app/components/VideoSection/VideoSection.tsx
Updated VIDEO_ID constant value; affects YouTube embed URL with no changes to component logic or control flow

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 A video swap, so quick and clean,
From one ID to a new scene,
YouTube links dance, pixels rearrange,
Just a string, yet everything's changed! 🎬

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive The description is incomplete. It covers the summary and test plan sections but is missing several required template sections: Related Issues, Type of Change, Screenshots, and Additional Notes. Fill in the missing template sections, particularly specify the Type of Change (appears to be a bug fix based on title), link any related issues, and provide any additional context.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating the video section with a new YouTube video ID, which is the sole modification in this pull request.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

@Kitenite
Kitenite merged commit 3208dca into main Jan 26, 2026
12 checks passed
@Kitenite
Kitenite deleted the kiet-ho/reading branch January 26, 2026 22:21
@github-actions

github-actions Bot commented Jan 26, 2026 •

Copy link
Copy Markdown
Contributor

🚀 Preview Deployment

🔗 Preview Links

Service Status Link
Neon Database (Neon) ✅ View Branch
Fly.io Electric (Fly.io) ✅ View App
Vercel API (Vercel) ✅ Open Preview
Vercel Web (Vercel) ✅ Open Preview
Vercel Marketing (Vercel) ✅ Open Preview
Vercel Admin (Vercel) ✅ Open Preview
Vercel Docs (Vercel) ✅ Open Preview

Preview updates automatically with new commits

This branch was previously deployed

1 inactive deployment
preview — 68ffc076 Deployed Jan 26, 2026 by Kitenite via Deploy Docs #1142
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.

1 participant