Page MenuHomePhabricator

[native] Pull `useShowAvatarActionSheet` out of `Edit*Avatar`
ClosedPublic

Authored by atul on Apr 18 2023, 2:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 4, 11:21 PM
Unknown Object (File)
Fri, Nov 1, 5:34 AM
Unknown Object (File)
Fri, Nov 1, 1:03 AM
Unknown Object (File)
Fri, Nov 1, 1:03 AM
Unknown Object (File)
Fri, Nov 1, 1:02 AM
Unknown Object (File)
Fri, Nov 1, 12:54 AM
Unknown Object (File)
Oct 3 2024, 12:08 AM
Unknown Object (File)
Sep 21 2024, 3:00 AM
Subscribers

Details

Summary

Pulling out some more logic that's in common between EditUserAvatar and EditThreadAvatar.

NOTE: One more quick refactor to tidy up editAvatarOptions and then we're gooing to move UserAvatar and ThreadAvatar "inside" Edit*Avatar components instead of just passing in React.Node.
Test Plan

Action sheet appears as it did previously + ESLint + Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable