-
Notifications
You must be signed in to change notification settings - Fork 170
VIDCS-3685: Connection Service integration #542
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
base: main
Are you sure you want to change the base?
Conversation
This application provides a completed version of the OpenTok [Basic Video Chat tutorial](https://tokbox.com/developer/tutorials/android/) for Android (differing only in some additional validation checks). Upon deploying this sample application, you should be able to have two-way audio and video communication using OpenTok. | ||
|
||
Main features: | ||
* Connect to an OpenTok session | ||
* Publish an audio-video stream to the session | ||
* Subscribe to another client's audio-video stream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we improve this README? I was expecting to find an introduction about the connection services and FCM (and how the app supports it).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I am working on it still. Should be pushed still today
|
||
private FcmEventSender() {} | ||
|
||
// IMPORTANT: This network operation should NOT run on the main thread! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume there is one or several blocking calls, that's why you create a new thread. That said, isn't there any really asynchronous alternative?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are now using ExecutorService for async calls
Does the microphone and camera capture work in background with the latest android SDKs? |
Fixed issues when rejecting incoming calls
Updated sample with two buttons, outgoing call and incoming call. When tapping one of them will notify an incoming or outgoing call to the Android Telecom framework. A ConnectionService will start and configure the IncomingCallConnection or OutgoingCallConnection. Both are going to be instances of the VonnageConnectionClass. The ConnectionService will start and stop a foreground service that will let recording audio and video in the background. Incoming calls will show a notification with accept and reject buttons, that will accept or reject the call. Once the call has been accepted the OpenTok session will start. When the user ends the call the Connection is destroyed, the foreground service stopped and the session cleared. It's possible to notify an incoming call with the simulate_incoming_call.sh script provided in the sample. When an incoming call arrives and the user presses the volume down button the ring will be muted. When another call (like a PSTN) interrupts the video call the publisher should stop publishing. For outgoing calls the flow is simpler, tap on the outgoing call button and the ConnectionService and Connection will be created and the UI updated accordingly. |
Translated literals to english Using getSimpleName for TAGs Added `ACTION_NOTIFY_INCOMING_CALL` for launching incoming calls from the push notification and MainActivity Removed static usages from PhoneAccountManager Removed `setCurrentConnection` from VonageManager
Updated the sample with an Unhold button. When there is an ongoing session running and the user receives an external call ConnectionService will put the session on hold. When the external call ends the user has to click on Unhold to reactivate the ConnectionService state and the audio will restart. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG
@goncalocostamendes We can go ahead and finish with this. 2.31.0 is out. |
Providing sample integrating a connection service with the vonage webrtc functionalities