-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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(ios/android/sdk): delegate transcription chunk received event #14516
feat(ios/android/sdk): delegate transcription chunk received event #14516
Conversation
7f2b755
to
655e643
Compare
@@ -255,6 +255,10 @@ protected void onReadyToClose() { | |||
finish(); | |||
} | |||
|
|||
protected void onTranscriptionChunkReceived(HashMap<String, Object> extraData) { | |||
JitsiMeetLogger.i("Transcription chunk received: " + extraData); |
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.
Don't put this here. Put it in the main activity, and commented out.
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 see that all the other events like CONFERENCE_TERMINATED or READY_TO_CLOSE are here and MainActivity extends JitsiMeetActivity.
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.
READY_TO_CLOSE is special because it terminates the activity. The rest shouldn't really be there...
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.
Can we leave it as it is right now and do a separate PR where we tackle all the events after this?
Co-authored-by: Saúl Ibarra Corretgé <s@saghul.net>
b6d9f8e
to
e4824b0
Compare
4c71b93
to
4587d93
Compare
…itsi#14516) * feat(subtitles): created separate helpers for delegating transcription chunks events on web and mobile
No description provided.