Skip to content

feat: twilio connector warm transfer#6283

Open
anunaym14 wants to merge 4 commits into
mainfrom
am/twilio-warm-transfer
Open

feat: twilio connector warm transfer#6283
anunaym14 wants to merge 4 commits into
mainfrom
am/twilio-warm-transfer

Conversation

@anunaym14

Copy link
Copy Markdown
Member

No description provided.

@anunaym14 anunaym14 force-pushed the am/twilio-warm-transfer branch from 4e0360e to 2d6b7d8 Compare July 1, 2026 13:13
@anunaym14 anunaym14 force-pushed the am/twilio-warm-transfer branch from 2d6b7d8 to 7897c11 Compare July 6, 2026 00:03
@anunaym14 anunaym14 marked this pull request as ready for review July 6, 2026 00:06
@anunaym14 anunaym14 requested a review from a team as a code owner July 6, 2026 00:06
@anunaym14 anunaym14 requested a review from cnderrauber July 6, 2026 00:06
devin-ai-integration[bot]

This comment was marked as resolved.

@longcw longcw left a comment

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.

lgtm, something nit:

Comment thread livekit-agents/livekit/agents/beta/workflows/warm_transfer.py Outdated
self, *, room_name: str, identity: str, room: rtc.Room
) -> None:
# optional dep; keep SIP path import-free
from twilio.rest import Client # type: ignore

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.

nit: ask user to install twilio in error message?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

handled!

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.

3 participants