Page MenuHomePhabricator

[keyserver] Extend ios notifs to handle macos
ClosedPublic

Authored by michal on Feb 24 2023, 9:43 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 8:47 AM
Unknown Object (File)
Mon, Apr 8, 3:48 AM
Unknown Object (File)
Fri, Apr 5, 2:42 PM
Unknown Object (File)
Fri, Apr 5, 2:34 PM
Unknown Object (File)
Mar 16 2024, 4:19 PM
Unknown Object (File)
Mar 12 2024, 10:18 AM
Unknown Object (File)
Mar 12 2024, 10:18 AM
Unknown Object (File)
Mar 7 2024, 2:02 PM
Subscribers

Details

Summary

We can reuse most of out already existing code for macOS notifs. This diff mostly just replaces codeVersion with full platformDetails so we can pass a platform and change for example the apn topic depending if we are sending the notifcation to the ios or macos.

Depends on D6891

Test Plan

Check if notifications still work on ios

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat added inline comments.
keyserver/src/push/send.js
177 ↗(On Diff #23070)

We construct this already on line 167. Instead of doing twice, maybe we should assign to a variable?

This revision is now accepted and ready to land.Feb 26 2023, 4:43 AM

Extract platformDetails variables.