Page MenuHomePhabricator

[native] Replace generic useSaveUserAvatar with new generic setUserAvatar
ClosedPublic

Authored by ashoat on May 29 2023, 5:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 10, 1:59 PM
Unknown Object (File)
Oct 27 2024, 8:19 AM
Unknown Object (File)
Oct 22 2024, 2:57 PM
Unknown Object (File)
Oct 3 2024, 6:14 AM
Unknown Object (File)
Sep 29 2024, 4:46 AM
Unknown Object (File)
Sep 29 2024, 4:46 AM
Unknown Object (File)
Sep 21 2024, 4:43 AM
Unknown Object (File)
Sep 17 2024, 10:30 PM
Subscribers

Details

Summary

In the last diff, I introduced a generic setUserAvatar in EditUserAvatarProvider. In a later diff, I'll add some logic that allows me to "override" the action that gets performed for a successful avatar selection, so that I can reuse it in the registration flow.

This code replaces useSaveUserAvatar with the generic utility, to make the codebase more consistent.

Depends on D8017

Test Plan

Test setting emoji avatar for user from iOS simulator

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

native/utils/avatar-utils.js
41–48 ↗(On Diff #27234)

I bring this back in D8019

This revision is now accepted and ready to land.May 30 2023, 9:13 AM