-
Notifications
You must be signed in to change notification settings - Fork 444
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
Don't try to get phone account if self managed when creating a connection #685
base: master
Are you sure you want to change the base?
Don't try to get phone account if self managed when creating a connection #685
Conversation
@donaldkwong hey man, why did you close this PR? |
Thanks @donaldkwong, can you add a little bit of documentation for this behaviour ? |
@manuquentin , what were you thinking of in terms of documentation? Changes to README.md or code comments? |
Can you put changes in the README please ? |
Honestly not sure what to write in the README since this feels more like a bug fix than a feature, but I added a description of the behavior. |
I don't have permission to merge this PR. Can someone do it for me? |
hey @donaldkwong is this going to help me to use some functions in android without calling |
We want to use CallKeep without requesting READ_PHONE_STATE permissions on Android, which should be possible if we're in self managed state. However, the connection creation code in
VoiceConnectionService
always tries to get the phone account, which requires that permission to be granted. This diff adds a check to see if we're in self managed mode and, if so, don't try to get the phone account.