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

requestPermission not showing any popup nor resolving the promise - swpush #2

Closed
jishnunbhattathiri opened this issue Aug 19, 2019 · 8 comments

Comments

@jishnunbhattathiri
Copy link

jishnunbhattathiri commented Aug 19, 2019

Hi, I tried running this code and in my browser (both chrome and firefox latest versions) I am not getting the notification permission popup. push.requestPermssion is hence never resolved and its not sending back the endpoint data. could you please help get it working?

@alcfeoh
Copy link
Owner

alcfeoh commented Aug 19, 2019

Did you use the PWA branch? https://github.com/alcfeoh/ng-weather/tree/pwa Also you'll have to use HTTPS (very likely) for the PWA behavior to work properly.

@jishnunbhattathiri
Copy link
Author

Yeah I used PWA branch, but haven't tried with https, let me try that. Thank you.

@jishnunbhattathiri
Copy link
Author

Tried with https as well, but no luck ( infact I tried deploying to a firebase hosting here).

@alcfeoh
Copy link
Owner

alcfeoh commented Aug 20, 2019

Did you start the local service for notifications on port 9020?

@jishnunbhattathiri
Copy link
Author

Yes I have. I made a simpler replica of the code here It worked when I served the dist with http-server and used push-server.js for notifications. But still its showing inconsistency. When I unregister service worker and removed the notification permission from chrome, and again visited, its not working, not even sw is registering.

I have opened an issue in stackoverflow as well here If you encounter any such issue or have any idea on the same, it would be very much helpful.
Thank you for your response again.

@alcfeoh
Copy link
Owner

alcfeoh commented Aug 21, 2019 via email

@alcfeoh
Copy link
Owner

alcfeoh commented Aug 21, 2019

Closing because it's not code related, more likely a Chrome issue.

@alcfeoh alcfeoh closed this as completed Aug 21, 2019
@jishnunbhattathiri
Copy link
Author

Thanks for your insights

Kamalen added a commit to Kamalen/ng-weather that referenced this issue Apr 26, 2024
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

No branches or pull requests

2 participants