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

Separate react native part from rest of Android code #137

Merged

Conversation

LinusU
Copy link
Contributor

@LinusU LinusU commented Dec 9, 2019

We want to use the CallKeep part from Flutter, with these changes I think that we should be able to do that ☺️

@LinusU
Copy link
Contributor Author

LinusU commented Dec 9, 2019

(crap, forgot to add Constants.java 😂)

edit: fixed

@LinusU LinusU force-pushed the separate-react-native branch from b282ab3 to 4ee21f7 Compare December 9, 2019 17:44
@manuquentin
Copy link
Contributor

Successfully tested on our app, thanks @LinusU !

@manuquentin manuquentin merged commit 59b2526 into react-native-webrtc:master Jun 4, 2020
@manuquentin
Copy link
Contributor

@LinusU after more investigation, it appear that this PR breaks the way that callkeep wakes up the application when we try to make a call from the native application.
The wakeUpApplication sends an event to the activity, but we should wakeup the application instead.
I'll make a PR to revert this method but it will break the integration with Flutter.

Do you have an idea how to deal with this issue in react-native and Flutter ?

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