Page MenuHomePhabricator

[lib] [refactor] [ENG-530] remove list of users string in reply
AcceptedPublic

Authored by benschac on May 3 2022, 3:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 12, 1:38 AM
Unknown Object (File)
Tue, Apr 9, 12:35 AM
Unknown Object (File)
Thu, Apr 4, 2:27 AM
Unknown Object (File)
Mar 18 2024, 1:34 PM
Unknown Object (File)
Mar 11 2024, 10:57 PM
Unknown Object (File)
Mar 11 2024, 8:14 PM
Unknown Object (File)
Feb 19 2024, 2:50 AM
Unknown Object (File)
Feb 19 2024, 1:54 AM

Details

Reviewers
ashoat
atul
Summary
Test Plan

No test plan, other than confirming the design doesn't have the functionality I'm removing. I did speak with ashoat about the hover functionality of the reply bubble where users' names would show. He was a bit on the fence about it. Either way, we have this commit in git history and can always grab it and re-introduce if needed.

Additionally, this shouldn't be landed until the stack is ready.

Diff Detail

Repository
rCOMM Comm
Branch
inline-sidebar-web-and-native-ENG-530
Lint
No Lint Coverage
Unit
No Test Coverage