Page MenuHomePhabricator

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

Authored by ginsu on Dec 15 2023, 12:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 20 2024, 12:11 AM
Unknown Object (File)
Feb 19 2024, 10:53 PM
Unknown Object (File)
Feb 19 2024, 9:37 PM
Unknown Object (File)
Feb 19 2024, 7:21 PM
Unknown Object (File)
Feb 19 2024, 11:31 AM
Unknown Object (File)
Jan 28 2024, 11:12 AM
Unknown Object (File)
Jan 19 2024, 7:22 AM
Unknown Object (File)
Jan 17 2024, 9:03 PM
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