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)
Thu, Apr 4, 9:10 PM
Unknown Object (File)
Thu, Apr 4, 9:10 PM
Unknown Object (File)
Thu, Apr 4, 9:09 PM
Unknown Object (File)
Thu, Apr 4, 9:01 PM
Unknown Object (File)
Feb 11 2024, 12:22 AM
Unknown Object (File)
Feb 11 2024, 12:21 AM
Unknown Object (File)
Feb 11 2024, 12:08 AM
Unknown Object (File)
Feb 10 2024, 10:45 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