Paths

Table of Contentst

[lib/native] change useGetAvatarForThread to useAvatarForThread
ClosedPublic
Actions

Authored by ginsu on Mar 30 2023, 11:09 PM.

Details

Reviewers

ashoat
atul

Commits

rCOMM48c14934ffd0: [lib/native] change useGetAvatarForThread to useAvatarForThread

Summary

changed hook name to improve readability and clarity

Depends on D7263

Test Plan

flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.Mar 30 2023, 11:09 PM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptMar 30 2023, 11:09 PM

ginsu mentioned this in D7262: [native] introude thread avatar component.Mar 30 2023, 11:11 PM

ginsu edited the summary of this revision. (Show Details)

ginsu added reviewers: ashoat, atul.

ginsu added a parent revision: D7263: [native] switch all thread avatars to use ThreadAvatar component instead of Avatar.

ginsu added a child revision: D7265: [native] improve avatar component prop types.Mar 30 2023, 11:17 PM

Harbormaster completed remote builds in B17858: Diff 24457.Mar 30 2023, 11:25 PM

ginsu requested review of this revision.Mar 30 2023, 11:25 PM

ashoat accepted this revision.Mar 31 2023, 5:31 AM

This revision is now accepted and ready to land.Mar 31 2023, 5:31 AM

rebase before landing

Harbormaster completed remote builds in B17896: Diff 24512.Mar 31 2023, 8:56 AM

Closed by commit rCOMM48c14934ffd0: [lib/native] change useGetAvatarForThread to useAvatarForThread (authored by ginsu). · Explain WhyMar 31 2023, 9:41 AM

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMM48c14934ffd0: [lib/native] change useGetAvatarForThread to useAvatarForThread.

Revision Contents
Changeset List

Path

Size

lib/

shared/

avatar-utils.js

6 lines

native/

components/

thread-avatar.react.js

4 lines

Diff 24457

View Options

lib/shared/avatar-utils.js