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

feat: terminate the SFT 1:1 call once the other person hangup the call - WPB-7152 #1772

Merged
merged 24 commits into from
Aug 9, 2024

Conversation

KaterinaWire
Copy link
Contributor

@KaterinaWire KaterinaWire commented Aug 7, 2024

StoryWPB-7152 [iOS] Add flag for customer specific 1:1 calls in federated environment

Issue

A follow up PR of #1742

When two participants are in a 1:1 call on SFT, if someone ends the call, the call will stay ongoing for the other person until timeout.
This is because we don't rely on the SFT list, but we have a subconversation that we use for calling.

To resolve that, we agreed on a short term approch which aims to :

  • Not kick out the participant from subconversation if he leaves the call (MLS)
  • Rely on SFT list by checking if the other participant has lost audio, this way the call will be terminated immediately instead of waiting for timeout for MLS and if there was only 1 participant left for Proteus.
  • Remove sub conversation after the 1:1 call is ended (MLS)

Testing

Setup conferenceCalling feature flag, so useSFTForOneToOneCalls is true.
Make a 1:1 call between users from different backends.

Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

UI accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Make sure you use the API for UI elements that support large fonts.
  • All colors are taken from WireDesign.ColorTheme or constructed using WireDesign.BaseColorPalette.
  • New UI elements have Accessibility strings for VoiceOver.

@KaterinaWire KaterinaWire changed the title feat: terminate the SFT 1:1 call once the other person hangup the call WPB-7152 feat: terminate the SFT 1:1 call once the other person hangup the call - WPB-7152 Aug 7, 2024
Copy link
Contributor

github-actions bot commented Aug 7, 2024

Test Results

5 313 tests   5 313 ✅  18m 15s ⏱️
  595 suites      0 💤
    3 files        0 ❌

Results for commit bf2fa0e.

♻️ This comment has been updated with latest results.

@datadog-wireapp
Copy link

datadog-wireapp bot commented Aug 7, 2024

Datadog Report

Branch report: feat/close-1-1-calls-WPB-7152
Commit report: 7fb9faa
Test service: wire-ios-mono

✅ 0 Failed, 5313 Passed, 0 Skipped, 57.26s Total Time

Copy link
Contributor

@David-Henner David-Henner left a comment

Choose a reason for hiding this comment

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

waiting to see the answers to the comments

@KaterinaWire KaterinaWire self-assigned this Aug 8, 2024
Co-authored-by: John Nguyen <28632506+johnxnguyen@users.noreply.github.com>
Base automatically changed from feat/1-1-calls-in-federated-environment to release/cycle-3.112 August 8, 2024 15:08
@echoes-hq echoes-hq bot added the echoes: product-roadmap Work aligned with the customer-announced roadmap, targeting a specific release date. label Aug 8, 2024
@KaterinaWire KaterinaWire merged commit f93f8b4 into release/cycle-3.112 Aug 9, 2024
10 checks passed
@KaterinaWire KaterinaWire deleted the feat/close-1-1-calls-WPB-7152 branch August 9, 2024 07:44
zenkins pushed a commit that referenced this pull request Aug 9, 2024
…l - WPB-7152 (#1772)

Co-authored-by: David-Henner <david.henner@wire.com>
Co-authored-by: John Nguyen <28632506+johnxnguyen@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
echoes: product-roadmap Work aligned with the customer-announced roadmap, targeting a specific release date. release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants