Actually send the notification to all macos devices. One things that is different is that we don't include text for alert notif so it doesn't display automatically by macos but we handle the visible notification ourselves. More information in ENG-3056.
tomek inka kamil ashoat
- rCOMM8a4bdc4cabdc: [keyserver] Send macos notifications
Tested with the next diff, if the notifications are sent correctly.
It looks like you're adding support for sending push notifs, but support is still missing for rescinds and for updateBadgeCount. Rescinds may be hard but updateBadgeCount should be easy.
Before landing, can you create Linear tasks for each?
|546 ↗||(On Diff #23071)|
Can you add a code comment explaining why we don't include body or sounds for macOS?