[native] Move `ThreadAvatar` "inside" `EditThreadAvatar`
ClosedPublic
Actions

Authored by atul on Apr 18 2023, 4:00 PM.

Details

Reviewers

ashoat
ginsu

Commits

rCOMMa51ab899c288: [native] Move `ThreadAvatar` "inside" `EditThreadAvatar`

Summary

Same as D7508, but for EditThreadAvatar instead of EditUserAvatar.

Depends on D7508

Test Plan

Things continue to look as expected:

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Apr 18 2023, 4:00 PM

Herald added a subscriber: tomek. · View Herald TranscriptApr 18 2023, 4:00 PM

atul edited the test plan for this revision. (Show Details)Apr 18 2023, 4:07 PM

Harbormaster completed remote builds in B18486: Diff 25317.Apr 18 2023, 4:17 PM

atul requested review of this revision.Apr 18 2023, 4:18 PM

atul added a child revision: D7510: [native] Introduce naive `useSelectFromGalleryAndUpdateUserAvatar` hook.Apr 18 2023, 6:43 PM

ashoat accepted this revision.Apr 19 2023, 6:04 AM

This revision is now accepted and ready to land.Apr 19 2023, 6:04 AM

Closed by commit rCOMMa51ab899c288: [native] Move `ThreadAvatar` "inside" `EditThreadAvatar` (authored by atul). · Explain WhyApr 19 2023, 9:28 AM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMa51ab899c288: [native] Move `ThreadAvatar` "inside" `EditThreadAvatar`.

Revision Contents
Changeset List

Path

Size

native/

avatars/

edit-thread-avatar.react.js

9 lines

chat/

settings/

thread-settings-avatar.react.js

6 lines

Diff 25356

View Options

native/avatars/edit-thread-avatar.react.js