transistorsoft / react-native-background-geolocation

Sophisticated, battery-conscious background-geolocation with motion-detection
http://shop.transistorsoft.com/pages/react-native-background-geolocation
MIT License
2.66k stars 426 forks source link

locationAuthorizationAlert not taking to App Settings #2202

Closed sainjay closed 1 week ago

sainjay commented 2 weeks ago

Your Environment

Expected Behavior

should go to app settings

image

Actual Behavior

Nothing is happening on clicking Settings

image

Steps to Reproduce

  1. Set the location permissions to Never in app settings
  2. Request the permissions using await BackgroundGeolocation.requestPermission()
  3. Getting this warning WARN [requestPermission] FAILURE: 2
  4. Alert is not doing anything on clicking the Settings button.

Context

Trying to get to settings to enable Location Permissions.

Debug logs

Logs ``` PASTE_YOUR_LOGS_HERE ```
christocracy commented 2 weeks ago

Nothing is happening on clicking Settings

Sorry, I missed this. I’ll look.

christocracy commented 2 weeks ago

I don’t have a problem in the demo app.

this is pure native code controlling this. It has nothing to do with JavaScript or react-native.

sainjay commented 1 week ago

@christocracy This is the bug i'm seeing from Xcode logs, can you check:

BUG IN CLIENT OF UIKIT: The caller of UIApplication.openURL(_:) needs to migrate to the non-deprecated UIApplication.open(_:options:completionHandler:). Force returning false (NO).
default 10:29:21.395272-0500    CareGiver   Not push traits update to screen for new style 2, <UIWindowScene: 0x1094be280> (E34CC39F-D34D-4682-8BE6-484DA37400E1)
default 10:29:21.397857-0500    CareGiver   _didHideAlertController: <UIAlertController: 0x1322bd800>
error   10:29:21.399029-0500    CareGiver   BUG IN CLIENT OF UIKIT: The caller of UIApplication.openURL(_:) needs to migrate to the non-deprecated UIApplication.open(_:options:completionHandler:). Force returning false (NO).
christocracy commented 1 week ago

See CHANGELOG

Screenshot 2024-11-08 at 10 33 17 AM
sainjay commented 1 week ago

Thanks, working after updating to latest version.