hicommonwealth / commonwealth

A platform for decentralized communities
https://commonwealth.im
GNU General Public License v3.0
67 stars 44 forks source link

Enable push notifications prompt #6655

Open sachben91 opened 7 months ago

sachben91 commented 7 months ago

Description

When enabling push notifications for the first time, users should do so from mobile-only. Once the user has enabled push notifications, they can modify their preferences on mobile and desktop.

Since its not possible for us to detect if the user has downloaded the pwa, we will prompt the user to download the pwa with a link on the push notification screen.

Feature specs

Image

Image

Designs

Figma

burtonator commented 4 months ago

Just FYI I don't know if we can implement this 'switch toggle' UI. The way notifications work is that we get a callback saying that we CAN enable notifications and then it has to be done via a button. I will investigate though and verify. We're limited what we can due to the security model done with PWAs for notifications.

burtonator commented 4 months ago

Also, I don't think we need this until we implement PWA notifications. This doesn't apply to in-app notifications I think.

sachben91 commented 4 months ago

Yes, these are for PWA notifications and it will be used by the user once they have enabled the notifications. They will use these toggles to manage their notification settings.

ForestMars commented 1 month ago

This issue needs an associated development branch / PR, kudasai