Page MenuHomePhabricator

[lib/native/web] convert appendSuffix to useAppendCommunitySuffix
ClosedPublic

Authored by ginsu on Dec 15 2023, 12:47 AM.
Tags
None
Referenced Files
F3402271: D10349.id34740.diff
Mon, Dec 2, 3:40 PM
F3401707: D10349.diff
Mon, Dec 2, 1:30 PM
Unknown Object (File)
Sun, Nov 24, 4:51 PM
Unknown Object (File)
Thu, Nov 21, 4:30 AM
Unknown Object (File)
Thu, Nov 21, 4:30 AM
Unknown Object (File)
Thu, Nov 21, 4:30 AM
Unknown Object (File)
Thu, Nov 21, 4:30 AM
Unknown Object (File)
Thu, Nov 21, 4:30 AM
Subscribers

Details

Summary

As I was working on the new navigation sidebar, I noticed that all uses of appendSuffix was being wrapped in a useMemo. To "improve the neighborhood" I converted appendSuffix into a hook and factored out the useMemo into the hook

Test Plan

flow and confirmed that there were no regressions with the community names in the community drawer

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable