Page MenuHomePhabricator

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

Authored by ginsu on Dec 15 2023, 12:47 AM.
Tags
None
Referenced Files
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
Unknown Object (File)
Oct 19 2024, 2:56 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
No Lint Coverage
Unit
No Test Coverage