Paths

Table of Contentst

Differential D7513

[native] Introduce `useRemoveThreadAvatar` hook
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F494952: Screen Recording 2023-04-18 at 11.19.16 PM.mov
	Apr 19 2023, 3:22 AM

Subscribers

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 25360

native/avatars/avatar-hooks.js

Loading...

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

Loading...