[native] Introduce `useRemoveThreadAvatar` hook
ClosedPublic
Actions

Authored by atul on Apr 19 2023, 3:19 AM.

Details

Reviewers

ashoat
ginsu

Commits

rCOMM7e902a758592: [native] Introduce `useRemoveThreadAvatar` hook

Summary

We introduce the useRemoveThreadAvatar hook so we can unset the user avatar. This is effectively D7512 but for threads.

Depends on D7512

Test Plan

Able to clear thread avatars:

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Apr 19 2023, 3:19 AM

Herald added a subscriber: tomek. · View Herald TranscriptApr 19 2023, 3:19 AM

atul published this revision for review.Apr 19 2023, 3:22 AM

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

Harbormaster completed remote builds in B18490: Diff 25321.Apr 19 2023, 3:37 AM

ashoat accepted this revision.Apr 19 2023, 1:20 PM

This revision is now accepted and ready to land.Apr 19 2023, 1:20 PM

Closed by commit rCOMM7e902a758592: [native] Introduce `useRemoveThreadAvatar` hook. · Explain WhyApr 19 2023, 4:28 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM7e902a758592: [native] Introduce `useRemoveThreadAvatar` hook.

Revision Contents
Changeset List

Path

Size

native/

avatars/

avatar-hooks.js

25 lines

edit-thread-avatar.react.js

10 lines

Diff 25321

View Options

native/avatars/avatar-hooks.js