Skip to content

Conversation

@Devansh-dvd
Copy link

@Devansh-dvd Devansh-dvd commented Jan 19, 2026

Related to #4891

Summary

Updated the YouTube video player on the roadmap page to show related videos only from the AsyncAPI channel instead of user based recommendations.

Changes Made

  • Modified the YouTube embed configuration in roadmap.tsx to include the rel=0 parameter
  • Ensures only AsyncAPI channel videos are recommended when the video is paused or ends

Summary by CodeRabbit

  • Bug Fixes
    • Updated YouTube video embedding to prevent related videos from displaying after playback ends.

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

@netlify
Copy link

netlify bot commented Jan 19, 2026

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 8d25f4d
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/696e6c637f76850007891b10
😎 Deploy Preview https://deploy-preview-4980--asyncapi-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 19, 2026

📝 Walkthrough

Walkthrough

The pull request modifies the YouTubeEmbed component on the roadmap page by appending the &rel=0 parameter to the start timestamp URL. This parameter disables the display of related video recommendations after the embedded video finishes playing.

Changes

Cohort / File(s) Summary
YouTube Embed Configuration
pages/roadmap.tsx
Added &rel=0 parameter to YouTubeEmbed appendSrc prop to prevent related video recommendations from displaying after playback completion

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

Poem

🐰 A rabbit hopped through the roadmap so bright,
One parameter added, the embed feels right,
No related videos will clutter the view,
Just focus and learning for me and for you!
A simple fix, yet effective and true! 🎬✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The pull request title 'chore: roadmap recommendations' is vague and does not clearly describe the specific technical change made. While it references the roadmap, it does not indicate that the change involves updating YouTube embed configuration to restrict recommended videos. Consider using a more specific title that describes the actual change, such as 'chore: restrict YouTube recommendations on roadmap page' or 'fix: prevent unrelated YouTube recommendations in roadmap embed'.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ 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.

@asyncapi-bot
Copy link
Contributor

asyncapi-bot commented Jan 19, 2026

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 37
🟢 Accessibility 98
🟢 Best practices 92
🟢 SEO 100
🔴 PWA 33

Lighthouse ran on https://deploy-preview-4980--asyncapi-website.netlify.app/

@codecov
Copy link

codecov bot commented Jan 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (49fb7df) to head (8d25f4d).

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #4980   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        22           
  Lines          796       796           
  Branches       146       146           
=========================================
  Hits           796       796           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: To Be Triaged

Development

Successfully merging this pull request may close these issues.

2 participants