Page MenuHomePhabricator

[lib] Migrate CreateThread/CreateSubThread notifTexts to use EntityText
ClosedPublic

Authored by ashoat on Feb 4 2023, 10:03 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 27, 4:29 PM
Unknown Object (File)
Thu, Jun 27, 4:29 PM
Unknown Object (File)
Thu, Jun 27, 4:28 PM
Unknown Object (File)
Thu, Jun 27, 4:24 PM
Unknown Object (File)
Wed, Jun 12, 10:30 AM
Unknown Object (File)
Tue, Jun 11, 8:34 AM
Unknown Object (File)
Fri, Jun 7, 9:16 PM
Unknown Object (File)
May 28 2024, 6:02 AM
Subscribers

Details

Summary

Note that this introduces a require cycle by having message specs import directly from notif-utils, which imports into them. I think it's fine in modern Node.js.

Depends on D6577

Test Plan

Generated CreateThread and CreateSubThread messages on a web client in a chat with a different test user that is logged in on the iOS simulator. Background the iOS app and make sure the notif looks right

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable