damus-io / damus

iOS nostr client
GNU General Public License v3.0
1.96k stars 288 forks source link

Merge notification settings and enable push notification filtering #2324

Open jb55 opened 5 days ago

jb55 commented 5 days ago

We could also just not send notifications depending on some setting associated with the device token. We just need to figure out where we would store that?

We could use the API, but then the relay would have to make a request to the API to get these settings. We really wouldn't want the push notification server to hit the api every time it needed to send a notification.

At this point we would need some kind of caching mechanism would could get out of date. This is horrendously ugly. Entitlements would make this much simpler.

jb55 commented 5 days ago

This seems like more of a pressing reason to get the entitlement @danieldaquino ? Otherwise our push design is much more complicated.