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

Always showing hang up button in VOIP calls #4567

Merged
merged 2 commits into from
Nov 29, 2021
Merged

Conversation

ouchadam
Copy link
Contributor

Fixes the symptom of #4144

  • Updates the VectorCallActivity to always show the hang up button during the initial phases of a VOIP call (Idle, Create Offer, matching the Dialing behaviour)

At least from the logs submitted in #4144 there was a native media player error which we're not able to propagate up to our CallActivity which means we end up stuck in the Idle call state. I can't seem to find a way to hook into player errors via the RemoteAudioTrack 🤔

The only way I've been able to reproduce this issue is by instantly recreating the call (Activity.recreate) when it starts.

BEFORE AFTER
before-stuck-voip after-stuck-voip

- fixes the CallActivity looking stuck with no way to exit the call
@github-actions
Copy link

Unit Test Results

  66 files  ±0    66 suites  ±0   57s ⏱️ -9s
135 tests ±0  135 ✔️ ±0  0 💤 ±0  0 ±0 
418 runs  ±0  418 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 258d64a. ± Comparison against base commit bdbbdb7.

Copy link
Member

@bmarty bmarty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but maybe @ganfra can double check here?

@bmarty bmarty requested a review from ganfra November 29, 2021 10:36
Copy link
Member

@ganfra ganfra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes I guess it's ok like that, thanks!

@bmarty bmarty merged commit df23cc4 into develop Nov 29, 2021
@bmarty bmarty deleted the feature/adm/stuck-voip branch November 29, 2021 15:24
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.

3 participants