HomePhabricator
Diffusion Comm c97d3b71cef5

[native] Introduce `useNativeSetThreadAvatar` and use throughout `native`

Description

[native] Introduce useNativeSetThreadAvatar and use throughout native

Summary:
Thread equivalent of D8341

setThreadAvatar previously had a call to displayFailureAlert, however, this function is only relevant on native since we don't display alerts on web.

As part of making the provider platform-agnostic, we "lift up" error handling/alert displaying to avatar-hooks.

We replace usages of setThreadAvatar on native with nativeSetUserAvatar.

Test Plan: The modified flows on native continue to work as expected.

Reviewers: ashoat, ginsu

Reviewed By: ashoat

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D8757

Details

Provenance
atulAuthored on Aug 8 2023, 5:14 PM
Reviewer
ashoat
Differential Revision
D8757: [native] Introduce `useNativeSetThreadAvatar` and use throughout `native`
Parents
rCOMMd1bdf88145aa: [landing] fix body scrolling when modal is active
Branches
Unknown
Tags
Unknown