Page MenuHomePhabricator
Differential D7512

[native] Introduce `useRemoveUserAvatar` hook
ClosedPublic
Actions

Authored by atul on Apr 18 2023, 7:48 PM.
Tags
None
Referenced Files
F1933241: D7512.diff
Sun, Jun 2, 10:11 AM
Unknown Object (File)
Thu, May 30, 11:52 AM
Unknown Object (File)
Thu, May 30, 11:52 AM
Unknown Object (File)
Sun, May 26, 6:08 AM
Unknown Object (File)
Sat, May 18, 3:50 AM
Unknown Object (File)
Thu, May 16, 9:47 AM
Unknown Object (File)
Mon, May 13, 12:13 PM
Unknown Object (File)
Sat, May 11, 4:00 PM
View All 57 Files
Subscribers
tomek

Details

Reviewers
ashoat
ginsu
Commits
rCOMM683fb7914ed5: [native] Introduce `useRemoveUserAvatar` hook
Summary

We introduce the useRemoveUserAvatar hook so we can unset the user avatar. Will introduce something similar for threads momentarily.

Depends on D7511

Test Plan

Able to clear user avatar:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul created this revision.Apr 18 2023, 7:48 PM
Herald added a subscriber: tomek. · View Herald TranscriptApr 18 2023, 7:48 PM
atul published this revision for review.Apr 18 2023, 7:50 PM
atul edited the summary of this revision. (Show Details)
atul edited the test plan for this revision. (Show Details)
atul added a parent revision: D7511: [native] Introduce naive `useSelectFromGalleryAndUpdateThreadAvatar` hook.
atul added inline comments.Apr 18 2023, 8:01 PM
native/avatars/avatar-hooks.js
327–333 ↗(On Diff #25320)

Here's how it looks on Android:

Screenshot_1681873252.png (2×1 px, 145 KB)

atul attached a referenced file: F494908: Screenshot_1681873252.png. (Show Details)Apr 18 2023, 8:01 PM
Harbormaster completed remote builds in B18489: Diff 25320.Apr 18 2023, 8:05 PM
atul added a child revision: D7513: [native] Introduce `useRemoveThreadAvatar` hook.Apr 18 2023, 8:19 PM
ashoat accepted this revision.Apr 19 2023, 6:20 AM
ashoat added inline comments.
native/avatars/avatar-hooks.js
327–333 ↗(On Diff #25320)

Thanks for including this

This revision is now accepted and ready to land.Apr 19 2023, 6:20 AM
Closed by commit rCOMM683fb7914ed5: [native] Introduce `useRemoveUserAvatar` 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: rCOMM683fb7914ed5: [native] Introduce `useRemoveUserAvatar` hook.

Revision Contents
Changeset List

PathSize
native/
avatars/
36 lines
10 lines

Diff 25359

View Options

native/avatars/avatar-hooks.js

Loading...
View Options

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

Loading...