Paths

Table of Contentst

[native] Introduce `useRemoveThreadAvatar` hook
ClosedPublic
Actions

Authored by atul on Apr 18 2023, 8:19 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Nov 6, 3:31 AM

	Unknown Object (File)
	Wed, Nov 6, 3:31 AM

	Unknown Object (File)
	Mon, Nov 4, 11:22 PM

	Unknown Object (File)
	Mon, Nov 4, 11:22 PM

	Unknown Object (File)
	Oct 27 2024, 2:14 AM

	Unknown Object (File)
	Oct 14 2024, 1:47 AM

	Unknown Object (File)
	Oct 3 2024, 12:09 AM

	Unknown Object (File)
	Oct 3 2024, 12:09 AM

View All 95 Files

Subscribers

tomek

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 18 2023, 8:19 PM

Herald added a subscriber: tomek. · View Herald TranscriptApr 18 2023, 8:19 PM

atul published this revision for review.Apr 18 2023, 8:22 PM

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

Harbormaster completed remote builds in B18490: Diff 25321.Apr 18 2023, 8:37 PM

ashoat accepted this revision.Apr 19 2023, 6:20 AM

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

Closed by commit rCOMM7e902a758592: [native] Introduce `useRemoveThreadAvatar` hook (authored by atul). · Explain WhyApr 19 2023, 9:28 AM

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