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

[FIX] Duplicated call onAnswer. New open method. #111

Merged
merged 3 commits into from
Sep 8, 2021
Merged

[FIX] Duplicated call onAnswer. New open method. #111

merged 3 commits into from
Sep 8, 2021

Conversation

efraespada
Copy link
Contributor

@efraespada efraespada commented Sep 6, 2021

  • Allows more flexible workflows for managing phone accounts without alerts.
  • Fixes duplicated call of onAnswer.
// always is called
void onAnswer(int videoState)
// and after:
void onAnswer() <-- now this is ignored

But the void onAnswer(int videoState) method (in VoiceConnection) stills being invoked twice. Before this change, CallKeepPerformAnswerCallAction was called x4.

  • Added support for drawable icons for the foreground service's notification:
'drawable/ic_notification_launcher'
'mipmap/ic_notification_launcher'
  • Readme updated

@efraespada efraespada changed the title feature: open method openPhoneAccounts [FIX] Duplicated call onAnswer. New open method. Sep 7, 2021
Copy link
Member

@cloudwebrtc cloudwebrtc left a comment

Choose a reason for hiding this comment

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

LGTM

@efraespada
Copy link
Contributor Author

Not merged yet

@cloudwebrtc cloudwebrtc merged commit dd1c872 into flutter-webrtc:master Sep 8, 2021
ejimenez93 pushed a commit to Griot-Inc/callkeep that referenced this pull request Sep 9, 2021
commit dd1c872
Author: Efra Espada <efraespada@gmail.com>
Date:   Wed Sep 8 12:23:41 2021 +0200

    [FIX] Duplicated call onAnswer. New open method. (flutter-webrtc#111)

    * feature: open method openPhoneAccounts

    * feature: fixed duplicated call onAnswer on VoiceConnection

    * feature: added support for drawables in notification icon in foreground service
ejimenez93 pushed a commit to Griot-Inc/callkeep that referenced this pull request Sep 9, 2021
…#1)

* feature: open method openPhoneAccounts

* feature: fixed duplicated call onAnswer on VoiceConnection

* feature: added support for drawables in notification icon in foreground service

Co-authored-by: Efra Espada <efraespada@gmail.com>
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.

2 participants