Page MenuHomePhabricator

[web] deprecate stringForReactionList
ClosedPublic

Authored by ginsu on Aug 2 2023, 12:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 1, 1:22 PM
Unknown Object (File)
Mon, Oct 28, 7:43 AM
Unknown Object (File)
Sep 28 2024, 9:32 AM
Unknown Object (File)
Sep 28 2024, 9:32 AM
Unknown Object (File)
Sep 28 2024, 9:32 AM
Unknown Object (File)
Sep 28 2024, 9:31 AM
Unknown Object (File)
Sep 28 2024, 9:23 AM
Unknown Object (File)
Sep 20 2024, 8:41 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
Lint Not Applicable
Unit
Tests Not Applicable

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.