Page MenuHomePhabricator

[lib] getKeyserverAdmin -> useKeyserverAdmin
ClosedPublic

Authored by ashoat on Jan 29 2023, 2:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 10, 4:26 AM
Unknown Object (File)
Sun, Nov 10, 4:26 AM
Unknown Object (File)
Sun, Nov 10, 4:24 AM
Unknown Object (File)
Tue, Nov 5, 2:09 AM
Unknown Object (File)
Oct 3 2024, 9:20 PM
Unknown Object (File)
Sep 28 2024, 7:29 PM
Unknown Object (File)
Sep 28 2024, 5:20 PM
Unknown Object (File)
Sep 27 2024, 10:51 AM
Subscribers

Details

Summary

This makes sense as a hook, and it will make it easier to incorporate useENSNames in the next diff.

Test Plan

Flow, and making sure CommunityPill on native & ChatThreadAncestors on web still look right (only two uses of this function)

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/ens
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

native/components/community-pill.react.js
22

I think it's okay to get rid of this useMemo since we're memoizing a primitive here, but if the CPU cost of community.members.find is judged too high I can bring it back

atul edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Jan 29 2023, 3:31 PM