Page MenuHomePhabricator

[lib] Allow multiple messages in CreateSidebar notificationTexts
ClosedPublic

Authored by ashoat on Feb 23 2023, 1:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 31, 8:13 AM
Unknown Object (File)
Sun, Mar 31, 8:13 AM
Unknown Object (File)
Sun, Mar 31, 8:12 AM
Unknown Object (File)
Sun, Mar 31, 8:06 AM
Unknown Object (File)
Feb 21 2024, 3:13 PM
Unknown Object (File)
Feb 21 2024, 9:08 AM
Unknown Object (File)
Feb 21 2024, 9:08 AM
Unknown Object (File)
Feb 21 2024, 9:08 AM
Subscribers

Details

Summary

In a future diff we introduce a notificationCollapseKey so that we have access to the SIDEBAR_SOURCE when generating the notif text for the CREATE_SIDEBAR mention.

Ahead of that, this diff updates the notificationTexts code so that it doesn't assert that there is only one message in the collapse result.

Depends on D6867

Test Plan

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