Skip to content

Fix broken modal.com URLs: blog typos and wrong doc paths#1503

Merged
charlesfrye merged 1 commit intomainfrom
devin/1771630577-fix-internal-link-typos
Feb 20, 2026
Merged

Fix broken modal.com URLs: blog typos and wrong doc paths#1503
charlesfrye merged 1 commit intomainfrom
devin/1771630577-fix-internal-link-typos

Conversation

@rchalamala
Copy link
Copy Markdown
Contributor

@rchalamala rchalamala commented Feb 20, 2026

Fixes 5 broken modal.com URLs across comments in example files. All currently 404; all fixes verified to return 200.

File Broken URL Fix Issue
ministral3_inference.py /blog/announcing-h200-b200 /blog/introducing-b200-h200 Wrong blog slug
vllm_inference.py /blog/announcing-h200-b200 /blog/introducing-b200-h200 Wrong blog slug
very_large_models.py /blog/introducting-b200-h200 /blog/introducing-b200-h200 Typo: "introducting"
sortformer2_1_speaker_diarization.py /docs/guide/asgi /docs/guide/webhooks Page moved
sandbox_agent.py /docs/images /docs/guide/images Missing /guide/ segment

No functional code changes — all edits are in comments only.

Link to Devin run | Requested by @rchalamala

Human Review Checklist

  • Verify the ASGI → webhooks change: the link text still says "ASGI" but the old /docs/guide/asgi page no longer exists. Confirm /docs/guide/webhooks is the correct destination for ASGI documentation.

Type of Change

  • Example updates (Bug fixes, new features, etc.)

Open with Devin

Co-Authored-By: Rahul Chalamala <rahul@modal.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@charlesfrye charlesfrye marked this pull request as ready for review February 20, 2026 23:44
@charlesfrye charlesfrye self-requested a review February 20, 2026 23:44
@charlesfrye
Copy link
Copy Markdown
Collaborator

@prbot approve

@modal-pr-review-automation
Copy link
Copy Markdown

@charlesfrye is not in the sudoers file. This incident will be reported.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review

# ## Using WebSockets to stream audio and diarization results

# We use a Modal [ASGI](https://modal.com/docs/guide/asgi) app to serve the diarization results
# We use a Modal [ASGI](https://modal.com/docs/guide/webhooks) app to serve the diarization results
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚩 ASGI link text doesn't match new webhooks destination

The link text still says [ASGI] but the URL was changed from /docs/guide/asgi to /docs/guide/webhooks. If the webhooks page doesn't cover ASGI specifics, this could be misleading to readers. The PR checklist notes this was reviewed and the old /docs/guide/asgi page no longer exists, so /docs/guide/webhooks is the closest valid destination. However, it may be worth updating the link text from ASGI to something like ASGI/webhooks to better match the destination page content.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

@charlesfrye
Copy link
Copy Markdown
Collaborator

@prbot approve rchalamala

@modal-pr-review-automation
Copy link
Copy Markdown

@charlesfrye is not in the sudoers file. This incident will be reported.

@charlesfrye
Copy link
Copy Markdown
Collaborator

@prbot approve @rchalamala

Copy link
Copy Markdown

@modal-pr-review-automation modal-pr-review-automation bot left a comment

Choose a reason for hiding this comment

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

Approved 👍. @rchalamala will follow-up review this.

@charlesfrye charlesfrye merged commit bdc5f06 into main Feb 20, 2026
10 checks passed
@charlesfrye charlesfrye deleted the devin/1771630577-fix-internal-link-typos branch February 20, 2026 23:48
Copy link
Copy Markdown
Contributor Author

@rchalamala rchalamala left a comment

Choose a reason for hiding this comment

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

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants