Skip to content

Add basic Twilio integration docs pointing to GitHub integration - #295

Open
J-Jaywalker wants to merge 8 commits into
mainfrom
add_twilio_integration_docs
Open

Add basic Twilio integration docs pointing to GitHub integration#295
J-Jaywalker wants to merge 8 commits into
mainfrom
add_twilio_integration_docs

Conversation

@J-Jaywalker

Copy link
Copy Markdown
Contributor

Adds a new integration guide walking developers through the Twilio ↔ Speechmatics reference proxy: what Twilio is, what the integration does, and how to get a bridged two-party call transcribed on their own machine.

  • New page docs/integrations-and-sdks/twilio.mdx — explains Media Streams and the audio-fork concept, includes a Mermaid architecture diagram, an interactive checkbox prerequisites list, and Python / Node.js install-and-run steps in language tabs. Ends with a troubleshooting table covering the four errors most users will hit (missing env vars, wrong number, trial-mode unverified caller, and "application error has occurred").
  • Sidebar docs/integrations-and-sdks/sidebar.ts — adds a Twilio entry after Vapi.
  • Overview docs/integrations-and-sdks/index.mdx — adds a new Telephony integrations section with a Twilio LinkCard.

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview Aug 3, 2026 3:13pm

Request Review

Comment thread docs/integrations-and-sdks/twilio.mdx Outdated

@petemomo petemomo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed against the docs style guide, terminology, and product-architecture. Strong how-to overall — clear flow, a Mermaid diagram with a prose equivalent, language tabs, and a solid troubleshooting table with real Twilio error codes. Comments inline (one blocking: the missing Twilio logo).

Optional nits (non-blocking):

  • μ-law / mulaw: the diagram says "μ-law", the prose says "mulaw" — pick one.
  • "per-speaker" vs "per-channel": this uses channel diarization (inbound/outbound channels), so "per-channel" is more precise than "per-speaker".
  • Trivia: double space in "hands that audio to Speechmatics with"; the banner/console output blocks have no language tag (the style guide wants one — text is fine); and the frontmatter description opens "Learn how to…" where the guide prefers an imperative.

Comment thread docs/integrations-and-sdks/index.mdx
Comment thread docs/integrations-and-sdks/twilio.mdx Outdated
cp .env.example .env
```

Open `.env` in your editor and the environment variables. The proxy won't run if any are missing.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Missing a verb here — should be something like: "Open .env in your editor and set the environment variables."

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added missing verb, "set"

Comment thread docs/integrations-and-sdks/twilio.mdx Outdated

This guide walks you through a small proxy that takes that audio stream and feeds it into Speechmatics Realtime, printing per-speaker transcripts to your console as the call unfolds. The [reference repo](https://github.com/speechmatics/speechmatics-twilio-integration) ships the same proxy in two languages — [Python](https://github.com/speechmatics/speechmatics-twilio-integration/tree/main/python) and [Node.js](https://github.com/speechmatics/speechmatics-twilio-integration/tree/main/js) — so pick whichever fits your team.

## How it works

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Heading nit: the style guide lists pronoun-dependent headings ("How this works") as invalid — name the subject, e.g. "How the integration works" or "How the proxy works".

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

changed to "how the proxy works"

Comment thread docs/integrations-and-sdks/twilio.mdx Outdated
@petemomo

petemomo commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

@J-Jaywalker Process note (not blocking): this PR and #291 (Zapier) both edit docs/integrations-and-sdks/index.mdx and docs/integrations-and-sdks/sidebar.ts. Whichever merges second will hit a conflict and need a rebase — worth coordinating the merge order.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants