Page MenuHomePhabricator

[web] deprecate stringForReactionList
ClosedPublic

Authored by ginsu on Aug 2 2023, 12:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 9, 11:46 PM
Unknown Object (File)
Sun, Jun 2, 10:58 PM
Unknown Object (File)
Sun, Jun 2, 10:57 PM
Unknown Object (File)
Sun, Jun 2, 10:57 PM
Unknown Object (File)
Sun, May 26, 10:06 AM
Unknown Object (File)
Sun, May 19, 8:00 PM
Unknown Object (File)
Sun, May 19, 8:00 PM
Unknown Object (File)
Sun, May 19, 8:00 PM
Subscribers

Details

Summary

In D8708 we got rid of the last usage of stringForReactionList. Now that this function is no longer being used we can get rid of this funciton and the unit tests associated with this function

Depends on D8708

Test Plan

flow and did a global search of stringForReactionList to confirm that this function was not being used anywhere else in the codebase

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu added reviewers: atul, kamil.
ginsu published this revision for review.Aug 2 2023, 12:25 PM
This revision is now accepted and ready to land.Aug 3 2023, 4:57 AM
This revision was automatically updated to reflect the committed changes.