Deprecate and remove "requestAuthorization"

[matt-oakes]

Ask your Question

We should deprecate and remove the requestAuthorization method from this library. This is currently a broken implementation as it does not allow you to specific what level of permissions you would like on iOS and doesn't do anything at all on Android.

I think that we should instead be asking users to use a dedicated API for managing permissions. react-native-permissions is a good example, but really React Native should have a cross platform API for this builtin, rather than just supporting Android through PermissionsAndroid.

[wbroek]

@matt-oakes do not agree that we need to remove the authorization. Yes you can use react-native-permissions which is great to do a check on forehand but I do not think you can do without it in the geolocation library because you need to have a status of the authorization anyway if a request to the api fails. You want to give back a good error message. Also when using a native geo api on iOS for example it will popup the message anyway and we need to take care of this scenario as wel in my opinion. We see the same with react-native-camera which has the authorization build in; as it should be in my opinion

[MrLoh]

Maybe you could go the same route as react native permission and provide a cocoa pods config that excludes the permission request, to let people use their preferred permission solution without requiring to add string to the Info plist for unrequested permission.