Page MenuHomePhabricator
Differential D7069

[native] render the rest of the user avatars
ClosedPublic
Actions

Authored by ginsu on Tue, Mar 14, 2:03 PM.
Tags
None
Referenced Files
F445168: D7069.id.diff
Sat, Mar 25, 2:52 PM
Unknown Object (File)
Fri, Mar 24, 10:12 PM
Unknown Object (File)
Tue, Mar 21, 12:32 AM
Unknown Object (File)
Sun, Mar 19, 5:27 PM
Unknown Object (File)
Sun, Mar 19, 5:27 PM
Unknown Object (File)
Sun, Mar 19, 12:43 PM
F427705: Screenshot 2023-03-14 at 5.14.01 PM.png
Tue, Mar 14, 2:14 PM
F427699: Screenshot 2023-03-14 at 5.08.50 PM.png
Tue, Mar 14, 2:14 PM
View All 12 Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
ashoat
atul
Commits
rCOMM7dd079eabea8: [native] render the rest of the user avatars
Summary

Render the rest of the user avatars. Since the logic for rendering the remaining user avatars in our app was pretty similar, I decided to package it all in one diff

Depends on D7057

Linear Task: https://linear.app/comm/issue/ENG-3107/build-out-a-dummy-avatar-component-and-render-it-everywhere-we-will

Test Plan

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

Message reactions modal:

Screenshot 2023-03-14 at 5.07.33 PM.png (1×1 px, 795 KB)

Thread settings member:

Screenshot 2023-03-14 at 5.07.49 PM.png (1×1 px, 990 KB)

Typeahead tooltip:

Screenshot 2023-03-14 at 5.08.06 PM.png (1×1 px, 1 MB)

User list user:

Screenshot 2023-03-14 at 5.09.25 PM.png (1×1 px, 737 KB)

Screenshot 2023-03-14 at 5.08.50 PM.png (1×1 px, 745 KB)

Relationship list item:

Screenshot 2023-03-14 at 5.14.01 PM.png (1×1 px, 732 KB)

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu created this revision.Tue, Mar 14, 2:03 PM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptTue, Mar 14, 2:03 PM
ginsu edited the summary of this revision. (Show Details)Tue, Mar 14, 2:14 PM
ginsu edited the test plan for this revision. (Show Details)
ginsu added reviewers: ashoat, atul.
ginsu added a parent revision: D7067: [native] render user avatars in chat screen.
ginsu attached a referenced file: F427705: Screenshot 2023-03-14 at 5.14.01 PM.png. (Show Details)
ginsu attached a referenced file: F427699: Screenshot 2023-03-14 at 5.08.50 PM.png. (Show Details)
ginsu attached a referenced file: F427700: Screenshot 2023-03-14 at 5.09.25 PM.png. (Show Details)
ginsu attached a referenced file: F427694: Screenshot 2023-03-14 at 5.08.06 PM.png. (Show Details)
ginsu attached a referenced file: F427689: Screenshot 2023-03-14 at 5.07.49 PM.png. (Show Details)
ginsu attached a referenced file: F427681: Screenshot 2023-03-14 at 5.07.33 PM.png. (Show Details)
Harbormaster completed remote builds in B17332: Diff 23738.Tue, Mar 14, 2:18 PM
ginsu requested review of this revision.Tue, Mar 14, 2:18 PM
ashoat accepted this revision.Wed, Mar 15, 1:53 PM

Seems uncontroversial

This revision is now accepted and ready to land.Wed, Mar 15, 1:53 PM
ginsu added a child revision: D7099: [native] add feature flag to avatars render work.Mon, Mar 20, 7:09 AM
ginsu updated this revision to Diff 23851.Mon, Mar 20, 7:14 AM

rebase after switching up diff stack order

ginsu edited the summary of this revision. (Show Details)Mon, Mar 20, 7:15 AM
ginsu added a parent revision: D7057: [native] render avatar in profile screen.
ginsu removed a parent revision: D7067: [native] render user avatars in chat screen.
Harbormaster completed remote builds in B17409: Diff 23851.Mon, Mar 20, 7:30 AM
ginsu updated this revision to Diff 23899.Mon, Mar 20, 3:25 PM

rebase before landing

Harbormaster failed remote builds in B17444: Diff 23899!Mon, Mar 20, 3:38 PM
ginsu updated this revision to Diff 23940.Tue, Mar 21, 8:52 AM

rebase after merge

Harbormaster completed remote builds in B17478: Diff 23940.Tue, Mar 21, 9:08 AM
Closed by commit rCOMM7dd079eabea8: [native] render the rest of the user avatars (authored by ginsu). · Explain WhyTue, Mar 21, 9:24 AM
This revision was automatically updated to reflect the committed changes.
ginsu added a commit: rCOMM7dd079eabea8: [native] render the rest of the user avatars.

Revision Contents
Changeset List

Diff 23945

View Options

native/chat/message-reactions-modal.react.js

Loading...
View Options

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

Loading...
View Options

native/chat/typeahead-tooltip.react.js

Loading...
View Options

native/components/user-list-user.react.js

Loading...
View Options

native/profile/relationship-list-item.react.js

Loading...