Details

Reviewers

ashoat
atul

Commits

rCOMMf55b55b3fef7: [native] render thread avatar in CHANNEL AVATAR thread settings section

Summary

Introduce CHANNEL AVATAR section in thread settings and render thread avatar in this new section

Depends on D7134

Test Plan

Please take a look at the screenshots to see the changes I made:

staff user:

Screenshot 2023-03-22 at 3.34.29 AM.png (1×1 px, 816 KB)

non staff user:

Screenshot 2023-03-22 at 3.34.54 AM.png (1×1 px, 611 KB)

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Mar 22 2023, 5:47 AM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptMar 22 2023, 5:47 AM

Harbormaster completed remote builds in B17498: Diff 23966.Mar 22 2023, 6:02 AM

ginsu requested review of this revision.Mar 22 2023, 6:02 AM

ginsu edited the summary of this revision. (Show Details)Mar 22 2023, 7:35 AM

ginsu edited the test plan for this revision. (Show Details)

ginsu added reviewers: ashoat, atul.

ginsu added a parent revision: D7134: [native] update message list header to new design.

ginsu attached a referenced file: F439887: Screenshot 2023-03-22 at 3.34.29 AM.png. (Show Details)

ginsu attached a referenced file: F439888: Screenshot 2023-03-22 at 3.34.54 AM.png. (Show Details)

update prop types for thread settings avatar

ginsu added a child revision: D7136: [native] render thread avatars in community drawer.Mar 22 2023, 7:54 AM

Harbormaster completed remote builds in B17504: Diff 23972.Mar 22 2023, 8:00 AM

ashoat accepted this revision.Mar 22 2023, 7:29 PM

ashoat added inline comments.

native/chat/settings/thread-settings-avatar.react.js
19 ↗	(On Diff #23972)	You could skip this memo and even improve performance slightly by just wrapping the whole component in `React.memo`

This revision is now accepted and ready to land.Mar 22 2023, 7:29 PM

address feedback

Harbormaster completed remote builds in B17641: Diff 24143.Mar 26 2023, 7:20 AM

rebase before landing

Harbormaster completed remote builds in B17691: Diff 24214.Mar 27 2023, 3:34 PM

Closed by commit rCOMMf55b55b3fef7: [native] render thread avatar in CHANNEL AVATAR thread settings section. · Explain WhyMar 27 2023, 3:44 PM

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMMf55b55b3fef7: [native] render thread avatar in CHANNEL AVATAR thread settings section.

[native] render thread avatar in CHANNEL AVATAR thread settings section
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 23966

native/chat/settings/thread-settings-avatar.react.js

native/chat/settings/thread-settings.react.js

[native] render thread avatar in CHANNEL AVATAR thread settings sectionClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 23966

native/chat/settings/thread-settings-avatar.react.js

native/chat/settings/thread-settings.react.js

[native] render thread avatar in CHANNEL AVATAR thread settings section
ClosedPublic
Actions

Revision Contents
Changeset List