-
Notifications
You must be signed in to change notification settings - Fork 216
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
Thank You #2039
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I've been using twilio-video for nearly 2.5 years now and I want to express my gratitude to those involved with this repo's development. It's been a sincere pleasure to work with this SDK. It's obvious that the devs who built this cared about ergonomics and scalability.
Here are a few things I really appreciate about twilio-video.
The decision to re-emit (nearly) all events at the room level
This wasn't necessary, but the individuals who decided to do this really understand good code and library consumption. This single decision allowed me to avoid callback nesting on multiple resources and instead allowed me to attach all of my event handlers at the top level. Thank you!
Extreme flexibility
connect
- twilio takes care of it.LocalTrack
constructor.trackName
.Efficiency
When tracks aren't visible, they're not rendered. When they become visible again, they're rendered - all automatically.
Features
I'm sad to see this product go. It's exceptional and you should be proud of it. Thank you for the hard work you did and for advocating for devs and ergonomics.
The text was updated successfully, but these errors were encountered: