Skip to content
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

iOS - getNotificationPermissionStatus() never returns #27

Closed
DuncanKelbie opened this issue Jul 29, 2020 · 4 comments · Fixed by #28
Closed

iOS - getNotificationPermissionStatus() never returns #27

DuncanKelbie opened this issue Jul 29, 2020 · 4 comments · Fixed by #28

Comments

@DuncanKelbie
Copy link
Contributor

DuncanKelbie commented Jul 29, 2020

On iOS if authorizationStatus value is provisional then the call to getNotificationPermissionStatus() never returns and my app is stuck waiting forever. Ideally I would like the PermissionStatus enum to be extended to include provisional so I can run different logic in that scenario.

It would be great if getNotificationPermissionStatus() always returns even if iOS adds more possible authorizationStatus values in future so that my app does not get stuck waiting forever.

@praveenbbhati
Copy link

praveenbbhati commented Aug 5, 2020

Hi @DuncanKelbie

I've tested the app after the updated version.

When I check for permission status it returns PermissionStatus.provisional and when I request for permission app doesn't show any popup and returns PermissionStatus.unknown. I've tested on a simulator.

@Vanethos Do you know about this issue? or it does not work on the simulator.

@DuncanKelbie
Copy link
Contributor Author

@praveenbbhati You're right, if the current status is PermissionStatus.provisional then no popup is shown when you request permission. This will happen whether on simulator or device. Needs fixing and thanks for pointing that out

@DuncanKelbie
Copy link
Contributor Author

Created a new issue #29

@DuncanKelbie
Copy link
Contributor Author

DuncanKelbie commented Aug 5, 2020

@Vanethos @praveenbbhati Created pull request #30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants