-
Notifications
You must be signed in to change notification settings - Fork 72
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
Fixes for iOS and fully working with Android (including Oreo, Pie, Q) #46
base: master
Are you sure you want to change the base?
Conversation
woffu
commented
Aug 20, 2019
- IOS, We fixed the issue with the RNBoundary.podspec
- Android, We fixed the issue with startService when the app is in background and correctly handling it using a startForegroundService with a binded notification.
Any news on the delivery date for the fix? |
When do you think this fix will be merged? |
cc @eddieowens |
@eddieowens is this going to be merged or should I use @woffu's fork? |
"name": "react-native-boundary", | ||
"version": "1.1.1", | ||
"description": "native geofencing and region monitoring", | ||
"name": "react-native-boundary-woffu", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@woffu revert name / repo changes.
Any news if this will be merged? |
@woffu Your fixes seem to work! Great stuff. |