Skip to content

Conversation

@asyncapi-bot
Copy link
Contributor

@asyncapi-bot asyncapi-bot commented Jan 31, 2026

Automated changes by create-pull-request GitHub action

Summary by CodeRabbit

  • New Features
    • Expanded the newsroom video library with multiple new video entries.

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

@netlify
Copy link

netlify bot commented Jan 31, 2026

Deploy Preview for asyncapi-website ready!

Name Link
🔨 Latest commit 32a6e56
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/697df12c5c542c00087b4297
😎 Deploy Preview https://deploy-preview-5048--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 31, 2026

📝 Walkthrough

Walkthrough

This pull request extends the newsroom videos configuration by appending 120 lines of new video entries to the existing JSON array in config/newsroom_videos.json. Each entry includes image_url, title, description, and videoId fields. No existing entries were modified or removed.

Changes

Cohort / File(s) Summary
Newsroom Videos Configuration
config/newsroom_videos.json
Added multiple new video entries to the JSON array, each containing image_url, title, description, and videoId fields. Changes are purely additive with no modifications to existing entries.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

autoapproved, autoupdate

Suggested reviewers

  • derberg
  • princerajpoot20
  • akshatnema
  • sambhavgupta0705
  • anshgoyalevil
  • Mayaleeeee
  • devilkiller-ag
  • vishvamsinh28
  • asyncapi-bot-eve

Poem

🐰 Hop, hop, hooray!
New videos join the array today,
Our newsroom shines with content bright,
A hundred twenty lines of pure delight! 🎬

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions updating both meetings.json and newsrooom_videos.json, but the raw summary shows only changes to config/newsroom_videos.json with no modifications to meetings.json. Update the title to accurately reflect that only newsroom_videos.json was modified, or verify that meetings.json changes are included in the PR. Also correct the typo 'newsrooom_videos.json' to 'newsroom_videos.json'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-meetings/00c9ed57f588b72346f7023d564a8f055f7e38d4

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.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 3

🤖 Fix all issues with AI agents
In `@config/newsroom_videos.json`:
- Around line 87-120: Fix the title typos and spacing: remove the stray space
before commas in the title values for videoId "m-aLlapR2Mg", "ncOFU__8hLM",
"7vvdmDVZPX0", "SaIPQaROWsc", and "xD5smwg6az8", and correct the misspelling
"Auguest" to "August" in the title for videoId "ncOFU__8hLM"; update the
corresponding "title" string values so they read e.g. "AsyncAPI Community WG
Meeting, Tuesday August 12th 2025" (and similarly for the other entries) with no
extra spaces before commas.
- Around line 116-120: There are two entries for "AsyncAPI Community WG Meeting
, Tuesday July 29th 2025" in newsroom_videos.json (one with videoId 7vvdmDVZPX0
and another with videoId xD5smwg6az8); confirm which recording is correct and
remove the duplicate entry (the object containing "videoId": "xD5smwg6az8",
"image_url": "https://i.ytimg.com/vi/xD5smwg6az8/hqdefault.jpg", "title":
"AsyncAPI Community WG Meeting , Tuesday July 29th 2025") if it’s unintended, or
merge/update the record to a single entry ensuring the correct videoId and
image_url are used.
- Around line 140-144: There are two entries with identical "title" "AsyncAPI
Design WG Meeting, Tuesday June 17th 2025" and the same "description"
referencing issue `#1935` but different "videoId" values ("0srKJiLCIUc" vs
"uVAurockaPc"); confirm whether these are distinct recordings and if not remove
the duplicate entry that matches that title/description (e.g., remove the object
with "videoId": "0srKJiLCIUc" shown in the diff), or if both are valid, update
one of them to include distinguishing metadata (e.g., date/time or “Part 1/Part
2”) so they are not identical, and ensure the surrounding JSON commas/structure
remain valid after the change.

Comment on lines +87 to +120
"image_url": "https://i.ytimg.com/vi/m-aLlapR2Mg/hqdefault.jpg",
"title": "AsyncAPI Community WG Meeting , Tuesday September 9th 2025",
"description": "https://github.com/asyncapi/community/issues/2060.",
"videoId": "m-aLlapR2Mg"
},
{
"image_url": "https://i.ytimg.com/vi/XjckzYopxoA/hqdefault.jpg",
"title": "AsyncAPI Mentorship Program Wrap-Up - 2024",
"description": "The livestream will serve as the official wrap-up of the AsyncAPI 2024 Mentorship Program. It's an opportunity for mentors and ...",
"videoId": "XjckzYopxoA"
},
{
"image_url": "https://i.ytimg.com/vi/ncOFU__8hLM/hqdefault.jpg",
"title": "AsyncAPI Community WG Meeting , Tuesday Auguest 12th 2025",
"description": "https://github.com/asyncapi/community/issues/2014.",
"videoId": "ncOFU__8hLM"
},
{
"image_url": "https://i.ytimg.com/vi/7vvdmDVZPX0/hqdefault.jpg",
"title": "AsyncAPI Community WG Meeting , Tuesday July 29th 2025",
"description": "https://github.com/asyncapi/community/issues/1988.",
"videoId": "7vvdmDVZPX0"
},
{
"image_url": "https://i.ytimg.com/vi/SaIPQaROWsc/hqdefault.jpg",
"title": "First Meeting of New Governance Board , Tuesday July 29th 2025",
"description": "https://github.com/asyncapi/community/issues/1989.",
"videoId": "SaIPQaROWsc"
},
{
"image_url": "https://i.ytimg.com/vi/xD5smwg6az8/hqdefault.jpg",
"title": "AsyncAPI Community WG Meeting , Tuesday July 29th 2025",
"description": "https://github.com/asyncapi/community/issues/1988.",
"videoId": "xD5smwg6az8"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Fix title typos and spacing inconsistencies.

Line 88, Line 100, Line 106, Line 112, and Line 118 contain an extra space before commas and a misspelling (“Auguest”). These are user-facing and should be corrected.

🧹 Proposed cleanup
-    "title": "AsyncAPI Community WG Meeting , Tuesday September 9th 2025",
+    "title": "AsyncAPI Community WG Meeting, Tuesday September 9th 2025",
@@
-    "title": "AsyncAPI Community WG Meeting , Tuesday Auguest 12th 2025",
+    "title": "AsyncAPI Community WG Meeting, Tuesday August 12th 2025",
@@
-    "title": "AsyncAPI Community WG Meeting , Tuesday July 29th 2025",
+    "title": "AsyncAPI Community WG Meeting, Tuesday July 29th 2025",
@@
-    "title": "First Meeting of New Governance Board , Tuesday July 29th 2025",
+    "title": "First Meeting of New Governance Board, Tuesday July 29th 2025",
@@
-    "title": "AsyncAPI Community WG Meeting , Tuesday July 29th 2025",
+    "title": "AsyncAPI Community WG Meeting, Tuesday July 29th 2025",
🤖 Prompt for AI Agents
In `@config/newsroom_videos.json` around lines 87 - 120, Fix the title typos and
spacing: remove the stray space before commas in the title values for videoId
"m-aLlapR2Mg", "ncOFU__8hLM", "7vvdmDVZPX0", "SaIPQaROWsc", and "xD5smwg6az8",
and correct the misspelling "Auguest" to "August" in the title for videoId
"ncOFU__8hLM"; update the corresponding "title" string values so they read e.g.
"AsyncAPI Community WG Meeting, Tuesday August 12th 2025" (and similarly for the
other entries) with no extra spaces before commas.

Comment on lines +116 to +120
{
"image_url": "https://i.ytimg.com/vi/xD5smwg6az8/hqdefault.jpg",
"title": "AsyncAPI Community WG Meeting , Tuesday July 29th 2025",
"description": "https://github.com/asyncapi/community/issues/1988.",
"videoId": "xD5smwg6az8"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

cat -n config/newsroom_videos.json | sed -n '100,125p'

Repository: asyncapi/website

Length of output: 1399


Confirm whether duplicate July 29th Community WG meeting entries are intentional.

Lines 104-109 and 116-121 both reference "AsyncAPI Community WG Meeting , Tuesday July 29th 2025" with the same issue (1988), but have different videoIds (7vvdmDVZPX0 vs xD5smwg6az8) and image URLs. If only one recording should be listed, remove the duplicate entry at lines 116-121.

🤖 Prompt for AI Agents
In `@config/newsroom_videos.json` around lines 116 - 120, There are two entries
for "AsyncAPI Community WG Meeting , Tuesday July 29th 2025" in
newsroom_videos.json (one with videoId 7vvdmDVZPX0 and another with videoId
xD5smwg6az8); confirm which recording is correct and remove the duplicate entry
(the object containing "videoId": "xD5smwg6az8", "image_url":
"https://i.ytimg.com/vi/xD5smwg6az8/hqdefault.jpg", "title": "AsyncAPI Community
WG Meeting , Tuesday July 29th 2025") if it’s unintended, or merge/update the
record to a single entry ensuring the correct videoId and image_url are used.

Comment on lines +140 to +144
{
"image_url": "https://i.ytimg.com/vi/0srKJiLCIUc/hqdefault.jpg",
"title": "AsyncAPI Design WG Meeting, Tuesday June 17th 2025",
"description": "https://github.com/asyncapi/community/issues/1935.",
"videoId": "0srKJiLCIUc"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

cat -n config/newsroom_videos.json | sed -n '130,150p'

Repository: asyncapi/website

Length of output: 1121


Remove duplicate "AsyncAPI Design WG Meeting, Tuesday June 17th 2025" entry.

Lines 141-144 duplicate lines 135-138 with identical title and description referencing the same GitHub issue #1935. While the video IDs differ (0srKJiLCIUc vs uVAurockaPc), having two entries with identical metadata for what appears to be the same meeting/topic is likely unintended. Confirm whether both represent distinct recordings that should both be listed, or remove the duplicate.

🤖 Prompt for AI Agents
In `@config/newsroom_videos.json` around lines 140 - 144, There are two entries
with identical "title" "AsyncAPI Design WG Meeting, Tuesday June 17th 2025" and
the same "description" referencing issue `#1935` but different "videoId" values
("0srKJiLCIUc" vs "uVAurockaPc"); confirm whether these are distinct recordings
and if not remove the duplicate entry that matches that title/description (e.g.,
remove the object with "videoId": "0srKJiLCIUc" shown in the diff), or if both
are valid, update one of them to include distinguishing metadata (e.g.,
date/time or “Part 1/Part 2”) so they are not identical, and ensure the
surrounding JSON commas/structure remain valid after the change.

@codecov
Copy link

codecov bot commented Jan 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (d57c7cf) to head (32a6e56).

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #5048   +/-   ##
=========================================
  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.

@asyncapi-bot
Copy link
Contributor Author

asyncapi-bot commented Jan 31, 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-5048--asyncapi-website.netlify.app/

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

Projects

Status: To Be Triaged

Development

Successfully merging this pull request may close these issues.

3 participants