Page MenuHomePhabricator

[keyserver] Update notif prepare functions to take bag of params
ClosedPublic

Authored by ashoat on Feb 23 2023, 1:16 PM.
Tags
None
Referenced Files
F1737310: D6872.id23199.diff
Fri, May 10, 11:35 PM
F1737308: D6872.id23032.diff
Fri, May 10, 11:34 PM
F1737307: D6872.id.diff
Fri, May 10, 11:34 PM
F1737306: D6872.diff
Fri, May 10, 11:34 PM
Unknown Object (File)
Thu, May 2, 11:02 PM
Unknown Object (File)
Thu, May 2, 11:02 PM
Unknown Object (File)
Thu, May 2, 10:12 PM
Unknown Object (File)
Thu, May 2, 8:56 PM
Subscribers

Details

Summary

These functions take a long list of inputs. I'm about to add one more, so I figured it would be best to refactor them to take an object of params instead of a list.

Depends on D6871

Test Plan

Flow. Also in combination with the rest of the stack, I confirmed that a sidebar started in response to a text message that @-mentions somebody will result in the @-mentioned user receiving a custom notif that ends with "that tagged you"

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable