Skip to content
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.

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

feat: Added documentation for twilio integration #125

Merged

Conversation

mahimairaja
Copy link
Contributor

@mahimairaja mahimairaja commented Mar 5, 2025

I have made changes to the Audio Streaming User Guide. Please have a looking, shall I add a demo under the demo directory?

I see that the readme is in the format of HF Model card? Should I repeat the same

this commit fixes #122

@mahimairaja
Copy link
Contributor Author

@freddyaboulton What do you think? Shall I add a complete demo usecase?

@freddyaboulton freddyaboulton marked this pull request as ready for review March 6, 2025 22:22
@freddyaboulton
Copy link
Owner

Sorry for the delay @mahimairaja ! I don't think we need to add a demo for this guide. But feel free to open a separate PR for a demo if you would like!

Thanks for the contribution. I have just edited the guide for style.

@freddyaboulton freddyaboulton merged commit f59e8c3 into freddyaboulton:main Mar 6, 2025
1 check failed
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.

[ New Demo ] Adding FastRTC Phone to a Twilio Phone Number
2 participants