Skip to content

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

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.

Sign up for GitHub

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

Merged
asyncapi-bot merged 1 commit into from
Aug 9, 2025
Merged

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

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -145,5 +145,12 @@
"url": "https://github.com/asyncapi/community/issues/2017",
"banner": "",
"date": "2025-08-07T16:00:00.000Z"
},
{
"title": "AsyncAPI Mentorship Program Wrap-Up",
"calLink": "https://www.google.com/calendar/event?eid=NXRqbzhsN3YxNmQwamxjOWNuODdhMTNzdDQgY19xOXRzZWlnbG9tZHNqNm5qdWh2YnB0czExY0Bn",
"url": "https://github.com/asyncapi/community/issues/2030",
"banner": "https://github.com/user-attachments/assets/a90d8599-6d77-4d0c-a7dd-14849c304c29",
"date": "2025-08-15T15:00:00.000Z"
}
]
12 changes: 6 additions & 6 deletions config/newsroom_videos.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,16 +17,16 @@
"description": "https://github.com/asyncapi/community/issues/1988.",
"videoId": "xD5smwg6az8"
},
{
"image_url": "https://i.ytimg.com/vi/FkGnFqMTj88/hqdefault.jpg",
"title": "AsyncAPI Community WG Meeting, Tuesday July 15th 2025",
"description": "https://github.com/asyncapi/community/issues/1969.",
"videoId": "FkGnFqMTj88"
},
Comment on lines +20 to +25
Copy link
Contributor

@coderabbitai coderabbitai bot Aug 9, 2025

Choose a reason for hiding this comment

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

💡 Verification agent

🧩 Analysis chain

New video entry added: validate thumbnail/videoId pairing and title punctuation consistency.

  • The image URL and videoId match (FkGnFqMTj88). Good.
  • Nitpick: titles across this file inconsistently place spaces before commas (e.g., "Meeting , Tuesday ..."). Consider standardizing to no space before commas for consistency.

Use this script to verify that each entry’s image_url (i.ytimg.com/vi//...) matches its videoId and to flag titles with a space before commas:

🏁 Script executed:

#!/bin/bash
set -euo pipefail

python - <<'PY'
import json, re, sys
from urllib.parse import urlparse

with open('config/newsroom_videos.json') as f:
    data = json.load(f)

problems = 0
for i, e in enumerate(data):
    vid = e.get('videoId', '')
    img = e.get('image_url', '')
    title = e.get('title', '')

    # Check i.ytimg.com videoId match
    if img and 'i.ytimg.com' in img:
        parts = urlparse(img).path.strip('/').split('/')
        # Expect ['vi', '<videoId>', 'hqdefault.jpg']
        if len(parts) >= 2 and parts[0] == 'vi':
            if parts[1] != vid:
                print(f"[mismatch] Entry {i}: image_url videoId '{parts[1]}' != videoId '{vid}'")
                problems += 1

    # Flag titles with space before comma
    if re.search(r'\s+,', title):
        print(f"[punctuation] Entry {i}: title has space before comma -> '{title}'")

# Specifically assert the new entry exists and is consistent
target = [e for e in data if e.get('videoId') == 'FkGnFqMTj88']
assert target, "New video entry with videoId FkGnFqMTj88 not found."
print("Check complete.")
sys.exit(problems)
PY

echo "HEAD request for the new thumbnail..."
curl -sI "https://i.ytimg.com/vi/FkGnFqMTj88/hqdefault.jpg" | head -n1 || true

Length of output: 558

Punctuation inconsistency in config/newsroom_videos.json

The new entry’s thumbnail/videoId pairing is correct (HTTP 200 on the HEAD request), but several titles across this file include a space before the comma. Please standardize by removing the space before each comma:

• Entry 0: “AsyncAPI Community WG Meeting , Tuesday July 29th 2025”
• Entry 1: “First Meeting of New Governance Board , Tuesday July 29th 2025”
• Entry 2: “AsyncAPI Community WG Meeting , Tuesday July 29th 2025”

Action: edit these titles (and any others) to use “…Meeting, Tuesday…” (no space before commas).

🤖 Prompt for AI Agents 
In config/newsroom_videos.json around lines 20 to 25, several video title
entries have an incorrect space before commas in the titles. Locate all titles
with a space before the comma, such as "Meeting , Tuesday", and remove the space
so it reads "Meeting, Tuesday" to ensure consistent punctuation formatting
throughout the file.

{
"image_url": "https://i.ytimg.com/vi/6AIUb1FNqEk/hqdefault.jpg",
"title": "AsyncAPI Community WG Meeting, Tuesday July 1st 2025",
"description": "https://github.com/asyncapi/community/issues/1962.",
"videoId": "6AIUb1FNqEk"
},
{
"image_url": "https://i.ytimg.com/vi/uVAurockaPc/hqdefault.jpg",
"title": "AsyncAPI Design WG Meeting, Tuesday June 17th 2025",
"description": "https://github.com/asyncapi/community/issues/1935.",
"videoId": "uVAurockaPc"
}
]
Loading