Enabling push notifications does not actually subscribe in some cases

Bug
1

During initial onboarding, I’ve observed that iOS users will install the forums as a PWA and then receive the notification consent banner when they first open the PWA.
If they click “enable notifications” in this consent banner, they are prompted for permissions and the confirmation notification is sent to them afterwards.

However, this does not actually subscribe users for push notifications, which is unexpected!

It appears that for the notification consent banner, the code here will go down the else path first to request user consent (Notification.requestPermission) but it will NOT subscribe to push notifications (subscribePushNotification).

The workaround at this point is for users to go to their notification preferences and click the “enable notifications” button there.

1 Like
2
3

Thanks so much for the detailed pull request here, will have someone from the team have a look over the next few weeks!

1 Like
5

Thank you!

I’ve done some more work on that PR since posting it - hopefully it won’t be too much :slight_smile:
I think this’ll substantially improve Discourse’s integration with WebPush, and it should give y’all an extension point that you can use to prod iOS users to install Discourse as a PWA.

1 Like