Page MenuHomePhabricator

[lib][keyserver] Convert notificationTexts to async functions
ClosedPublic

Authored by ashoat on Feb 3 2023, 4:14 PM.
Tags
None
Referenced Files
F3504398: D6571.diff
Fri, Dec 20, 8:53 AM
Unknown Object (File)
Sat, Dec 14, 1:30 PM
Unknown Object (File)
Sat, Dec 14, 1:30 PM
Unknown Object (File)
Sat, Dec 14, 1:30 PM
Unknown Object (File)
Sat, Dec 14, 1:30 PM
Unknown Object (File)
Sat, Dec 14, 1:22 PM
Unknown Object (File)
Fri, Dec 6, 6:13 PM
Unknown Object (File)
Nov 18 2024, 7:32 PM
Subscribers

Details

Summary

This will allow us to handle ENS resolution there. We'll probably actually do that outside of notificationTexts (probably in fullNotifTextsForMessageInfo), but it's probably good to make notificationTexts async as well (for eg. @ginsu's planned work on reactions notifs).

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable