Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update meetings.json and newsrooom_videos.json #3432

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions config/meetings.json
Original file line number Diff line number Diff line change
Expand Up @@ -138,5 +138,12 @@
"url": "https://github.com/asyncapi/community/issues/1596",
"banner": "https://github.com/user-attachments/assets/ff469484-a2c5-4aa2-8652-782f017947fb",
"date": "2024-11-26T16:00:00.000Z"
},
{
"title": "AsyncAPI Mentorship Program Kick-Off",
"calLink": "https://www.google.com/calendar/event?eid=aGg5cGtwdGV0YmpzbnBlcmdwYjFqMjU1cTQgY19xOXRzZWlnbG9tZHNqNm5qdWh2YnB0czExY0Bn",
"url": "https://github.com/asyncapi/community/issues/1599",
"banner": "https://github.com/user-attachments/assets/739381b2-b7de-4e08-b845-04e5a07ba7e0",
"date": "2024-12-03T13:00:00.000Z"
}
]
12 changes: 6 additions & 6 deletions config/newsroom_videos.json
Original file line number Diff line number Diff line change
@@ -1,4 +1,10 @@
[
{
"image_url": "https://i.ytimg.com/vi/XirMXiBNaBM/hqdefault.jpg",
"title": "Community Meeting, 16:00 UTC Tuesday November 26th 2024",
"description": "",
"videoId": "XirMXiBNaBM"
Comment on lines +3 to +6
Copy link

Choose a reason for hiding this comment

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

💡 Codebase verification

Future community meeting video entry needs attention

The video entry appears to be a placeholder for an upcoming community meeting (November 26th, 2024). The video ID is only referenced in this configuration file and follows these concerning patterns:

  • The meeting date is in the future
  • The video ID appears to be a placeholder as it's not yet accessible
  • Other entries in the file are for existing, published videos

Please ensure one of the following:

  • Remove this entry until the actual video is published and has a valid video ID
  • If this is meant as a placeholder, consider moving it to a separate configuration for upcoming events
🔗 Analysis chain

Verify the YouTube video accessibility

The video appears to be for a future community meeting. Let's ensure the video ID is properly configured.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Verify if the YouTube video ID exists and is accessible
# Expected: If the video is properly configured, the API should return valid data

curl -s "https://www.googleapis.com/youtube/v3/videos?id=XirMXiBNaBM&part=status" \
  -H "Accept: application/json" | jq .

Length of output: 645


Script:

#!/bin/bash
# Let's check if this video ID appears in other configuration files or documentation
# This might help us understand if it's a placeholder or a real video ID

rg -l "XirMXiBNaBM"

# Let's also check the structure of the entire newsroom_videos.json to see if there are any patterns
# or similar entries we can compare against

cat config/newsroom_videos.json

Length of output: 1246

},
{
"image_url": "https://i.ytimg.com/vi/Q2cvxsUUgzA/hqdefault.jpg",
"title": "AsyncAPI-Powered Event Feeds: 3 Steps to Streaming",
Expand All @@ -22,11 +28,5 @@
"title": "AsyncAPI and DDD - A Pretty Couple - Dr. Annegret Junker",
"description": "",
"videoId": "lCTdXmougTk"
},
{
"image_url": "https://i.ytimg.com/vi/b_BStEcLNqg/hqdefault.jpg",
"title": "Sustaining Open Source Beyond Code: The Importance of Marketing and Funding - Oluwabamikemi Kayode",
"description": "",
"videoId": "b_BStEcLNqg"
}
]