Skip to content

Broken internal link on Realtime API category page#2680

Merged
marklysze merged 2 commits intomainfrom
elva/fix-b84ef180
Apr 20, 2026
Merged

Broken internal link on Realtime API category page#2680
marklysze merged 2 commits intomainfrom
elva/fix-b84ef180

Conversation

@VasiliyRad
Copy link
Copy Markdown
Contributor

Files changed:

  • website/docs/_blogs/2025-01-08-RealtimeAgent-over-websocket/index.mdx

Took me two passes to get this right — first attempt missed the mark.

What I checked before submitting:

The round-2 fix correctly addresses the broken internal link as requested in the round-1 review notes.

What changed: /docs/blog/2024-12-20-RealtimeAgent/index/latest/docs/blog/2024/12/20/RealtimeAgent/

Verification:

  • New URL https://docs.ag2.ai/latest/docs/blog/2024/12/20/RealtimeAgent/ returns HTTP 200 ✅
  • Change is minimal and surgical — only the broken link is updated, no other content is altered ✅
  • Follows the explicit guidance from round 1 review notes (add /latest/ prefix, use date-segmented path) ✅

Note for human reviewer: The /latest/ prefix hardcoding is a reasonable pragmatic fix for this doc site's routing. If the versioning scheme ever changes, this link may need revisiting — but it unambiguously resolves today and is far better than the previous 404.


Like a river carves its path — small, steady changes shape the landscape. Fixed by Elva.

Internal link to RealtimeAgent blog post missing /latest/ prefix returns 404.

Bug: file-b84ef180
Fixed by Elva.
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 15, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ marklysze
❌ Elva


Elva seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Apr 15, 2026
Copy link
Copy Markdown
Collaborator

@marklysze marklysze left a comment

Choose a reason for hiding this comment

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

@VasiliyRad and Elva, please note the correction made.

@marklysze marklysze enabled auto-merge April 20, 2026 01:16
@marklysze marklysze disabled auto-merge April 20, 2026 01:16
@marklysze marklysze merged commit ec42ede into main Apr 20, 2026
21 of 22 checks passed
@marklysze marklysze deleted the elva/fix-b84ef180 branch April 20, 2026 01:16
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 92 files with indirect coverage changes

🚀 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

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants