-
-
Notifications
You must be signed in to change notification settings - Fork 361
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
Expand Diagnostic.permissionStatus to feature more permission statuses #443
Comments
HI, I've encountered an issue in building my IOS app when using the 6.0.4 version of the cordova-diagnostic-plugin.
Is this somehow related to this issue? When I revert back to version 6.0.2 the ios app builds just fine. However, my android app fails building when using version 6.0.2. Any help would be appreciated. |
Which version of XCode are you using? I wonder if older versions don't support this (https://developer.apple.com/documentation/usernotifications/unauthorizationstatus/ephemeral) I had no issues building with XCode 12 or 13. |
@matt-auckland |
If possible I would try building with a newer version of XCode, I think that support for the "Emphermeral Status" was added in XCode 12. |
So in #437 I added in the
AUTHORIZATION_PROVISIONAL
andAUTHORIZATION_EPHEMERAL
consts into the iOS side of the plugin, and I wonder if I should update theDiagnostic.permissionStatus
object in the JS side of the plugin to include these?I imagine there may be some other forms of status that could also be included, e.g. Precise Location and Approximate Location.
I'm happy to do the work and add all of these @dpa99c, just wondering if you had suggestions as to what other statuses are missing or if you think some shouldn't be added for whatever reason.
The text was updated successfully, but these errors were encountered: