[native] move navigate to emoji thread avatar creation function to be within edit thread avatar
ClosedPublic
Actions

Authored by ginsu on Apr 27 2023, 5:07 AM.

Details

Reviewers

ashoat
atul

Commits

rCOMMf65bfddfcd95: [native] move navigate to emoji thread avatar creation function to be within…

Summary

Small cleanup diff, I figured it would be better to have this logic within EditThreadAvatar

Depends on D7643

Test Plan

flow still able to navigate to the user emoji avatar creation screen

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Apr 27 2023, 5:07 AM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptApr 27 2023, 5:07 AM

ginsu edited the summary of this revision. (Show Details)Apr 27 2023, 5:08 AM

ginsu added reviewers: ashoat, atul.

ginsu added a parent revision: D7643: [native] have upload thread avatar processing/uploading loader only appear for thread being updated.

Harbormaster completed remote builds in B18832: Diff 25800.Apr 27 2023, 5:24 AM

ginsu requested review of this revision.Apr 27 2023, 5:24 AM

ginsu added a child revision: D7645: [native] improve factoring out redundant logic for saving an image thread avatar.Apr 27 2023, 5:41 AM

ashoat accepted this revision.Apr 27 2023, 5:25 PM

This revision is now accepted and ready to land.Apr 27 2023, 5:25 PM

rebase

Harbormaster completed remote builds in B18928: Diff 25927.Apr 28 2023, 8:27 PM

rebase before landing

Harbormaster completed remote builds in B18945: Diff 25944.May 1 2023, 5:07 AM

Closed by commit rCOMMf65bfddfcd95: [native] move navigate to emoji thread avatar creation function to be within…. · Explain WhyMay 1 2023, 5:17 AM

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMMf65bfddfcd95: [native] move navigate to emoji thread avatar creation function to be within….

Revision Contents
Changeset List

Path

Size

native/

avatars/

edit-thread-avatar.react.js

21 lines

chat/

settings/

thread-settings-avatar.react.js

20 lines

Diff 25800

View Options

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