urbanairship / urbanairship-cordova

Urban Airship integration with Apache Cordova
https://docs.airship.com/platform/mobile/setup/sdk/cordova/
Other
142 stars 149 forks source link

iOS location issue on uploading the appstore - ask for NSLocationAlwaysUsageDescription #297

Closed yoavpa-tr closed 5 years ago

yoavpa-tr commented 5 years ago

Hi, I got the following message from apple once I uploaded the app to the app store.

Missing Purpose String in Info.plist File - Your app's code references one or more APIs that access sensitive user data. The app's Info.plist file should contain a NSLocationAlwaysUsageDescription key with a user-facing purpose string explaining clearly and completely why your app needs the data. Starting Spring 2019, all apps submitted to the App Store that access user data will be required to include a purpose string. If you're using external libraries or SDKs, they may reference APIs that require a purpose string. While your app might not use these APIs, a purpose string is still required. You can contact the developer of the library or SDK and request they release a version of their code that doesn't contain the APIs. Learn more (https://developer.apple.com/documentation/uikit/core_app/protecting_the_user_s_privacy).Thanks so much for filing an issue or feature request! Please fill out the following (wherever relevant):

Does your plugin use the device location / uses an API that uses the device location?

Reproduction

I don't get any error once I build and run the app in debug/release mode. only got the above mail from apple once I uploaded the app store.

Additional Information

BrianBatchelder commented 5 years ago

@yoavpa This is a known issue. We are working on an update. In the meantime, if you add the strings as explained in our docs, Apple should accept your app.

crow commented 5 years ago

As of urbanairship-cordova release 8.0.0 - AirshipLocationKit has been broken out into its own module.

This module can be installed and linked via the airship-location-cordova plugin available here.