-
Notifications
You must be signed in to change notification settings - Fork 11
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
Implement 1:1 Calls #209
base: main
Are you sure you want to change the base?
Implement 1:1 Calls #209
Conversation
looks like flutter_webrtc package is causing integration test to fail, even if it isnt being used in the actual app. Going to have to disable integration test for now, and investigate this issue later |
@Airyzz, this is really cool, is there any ETA for this? |
Thanks! No ETA at the moment as I need to evaluate what changes I need to make for MatrixRTC support |
this is immensely epic |
@Airyzz as MatrixRTC is still in the making, I would recommend adding calls using the current spec (probably behind a labs switch, or a notification when first used), for multiple reasons:
(Sorry if my comment is dumb, I haven't yet catched up with the MatrixRTC spec, and don't exactly know what is included in it and what is not, neither the differences from the current VoIP spec.) |
@notramo I think its a good point - it does still need evaluating though because I believe the matrix SDK has had some changes since I last worked on this |
No description provided.