cybex-dev / twilio_voice

Flutter Twilio Voice Plugin
https://twilio-voice-web.web.app/
MIT License
39 stars 80 forks source link

Bug ios: aborting call before answer will not interrupt it #111

Closed eugenioadapta closed 1 year ago

eugenioadapta commented 1 year ago

Steps to reproduce: Place a call to any number. Close the call (hangup) before answer Issue: under IOS, the call would not be aborted

Expected behavior: the call should be interrupted

Possible solution: This line in https://github.com/diegogarciar/twilio_voice/blob/master/ios/Classes/SwiftTwilioVoicePlugin.swift at line 204 if (self.call != nil && self.call?.state == .connected) { should just become if (self.call != nil) {

Using a fork of this project with the proposed solution fixes the issue

cybex-dev commented 1 year ago

I should have such a fix added in my own fork, I'll create a PR to include this if you want me to.

eugenioadapta commented 1 year ago

@cybex-dev thanks, but my workaround is stil not tested 100%. Maybe we can experience problems when caller and receiver close the call at the same time? Not sure. Too bad that this project is abandoned. If you include this fix and publish your package to pub.dev let me know, we will include it in our app

cybex-dev commented 1 year ago

@cybex-dev thanks, but my workaround is stil not tested 100%. Maybe we can experience problems when caller and receiver close the call at the same time? Not sure. Too bad that this project is abandoned. If you include this fix and publish your package to pub.dev let me know, we will include it in our app

Sounds good - got a bunch of fixes, additions, feature updates, etc that I need to add - just haven't had the time.

I'll check in sometime in the next week or so.

cybex-dev commented 1 year ago

@eugenioadapta seems you aren't the only one to experience this issue.

I'm upgrading this library atm, will add this fix you suggested.

cybex-dev commented 1 year ago

@eugenioadapta I'll be reviewing this suggestion later today.

yacoublambaz commented 1 year ago

Any updates here? I'm facing the same issue.

cybex-dev commented 1 year ago

Hi @yacoublambaz - I'm actively making upgrades this week to merge new features, platform support, fixes, etc.

yacoublambaz commented 1 year ago

@cybex-dev I appreciate your efforts! To confirm, should I expect this bug to be fixed with your upgrades?

cybex-dev commented 1 year ago

I've yet to review all PRs and address all outstanding bugs/issues, though I will make a pre-release available for testing prior to pub.dev release (by @diegogarciar)

cybex-dev commented 1 year ago

Steps to reproduce: Place a call to any number. Close the call (hangup) before answer Issue: under IOS, the call would not be aborted

Expected behavior: the call should be interrupted

Possible solution: This line in https://github.com/diegogarciar/twilio_voice/blob/master/ios/Classes/SwiftTwilioVoicePlugin.swift at line 204 if (self.call != nil && self.call?.state == .connected) { should just become if (self.call != nil) {

Using a fork of this project with the proposed solution fixes the issue

Great suggestion, expect it in this upcoming release - if you like to test this early in this package, I'll be make a pre-release branch available before it goes live.

cybex-dev commented 1 year ago

Available in next release, closing.