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)
Sat, Feb 15, 9:30 PM
Unknown Object (File)
Thu, Feb 13, 1:27 PM
Unknown Object (File)
Thu, Feb 13, 1:27 PM
Unknown Object (File)
Thu, Feb 13, 1:20 PM
Unknown Object (File)
Thu, Feb 6, 5:16 AM
Unknown Object (File)
Tue, Feb 4, 5:46 AM
Unknown Object (File)
Mon, Feb 3, 10:48 AM
Unknown Object (File)
Thu, Jan 30, 8:23 PM

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