Page MenuHomePhabricator

[native] Update `useRemoveUserAvatar` to return `isLoading`
ClosedPublic

Authored by atul on Apr 19 2023, 11:28 AM.
Tags
None
Referenced Files
F3540113: D7528.diff
Thu, Dec 26, 4:31 AM
Unknown Object (File)
Thu, Dec 5, 2:31 AM
Unknown Object (File)
Thu, Nov 28, 4:44 AM
Unknown Object (File)
Nov 24 2024, 10:22 AM
Unknown Object (File)
Nov 24 2024, 8:33 AM
Unknown Object (File)
Nov 4 2024, 11:23 PM
Unknown Object (File)
Nov 1 2024, 9:13 AM
Unknown Object (File)
Nov 1 2024, 9:13 AM
Subscribers

Details

Summary

We modify useRemoveUserAvatar to return a tuple of useRemoveUserAvatar, isLoading.

In subsequent diff will consume this along w/ isLoading from useSelectFromGalleryAndUpdateUserAvatar` to render a spinner over the image when operation is in progress.

For convenience, I'm going to have a multiple diffs where I update the EditUserAvatar flow, and then a single diff where I update the EditThreadAvatar flow to match. There's a lot of repetition, but I have ideas on how to consolidate things once we have a shippable experience.


Depends on D7527

Test Plan

Consume inProgress in EditUserAvatar and log the value. The value being printed matched up with the progress of the flow.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable