breautek commented on PR #225:
URL: 
https://github.com/apache/cordova-plugin-geolocation/pull/225#issuecomment-1720403548

   Apple states:
   
   > If the user denies your app's use of the location service, this method 
reports a 
[kCLErrorDenied](https://developer.apple.com/documentation/corelocation/clerror/kclerrordenied?language=objc)
 error. Upon receiving such an error, you should stop the location service.
   
   
https://developer.apple.com/documentation/corelocation/cllocationmanagerdelegate/1423786-locationmanager?language=objc
   
   And Apple can be notoriously picky when it comes to app reviews, which makes 
me hesitant about merging this change.
   
   A simple workaround is to just hook into the `resume` event and resume 
watching which is what Apple would expect you to do natively in this situation.
   
   Not stopping the location manager means you're wastefully draining battery 
resources on the device when you **know** you won't get any GPS updates.
   
   So my apologies, I don't think this PR can be accepted.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to