Page MenuHomePhabricator

Move `native`-specific `uploadSelectedMedia` from `updateImageUserAvatar` to `nativeUpdateUserImageAvatar`
ClosedPublic

Authored by atul on Jun 27 2023, 6:16 PM.
Tags
None
Referenced Files
F3518227: D8346.diff
Sun, Dec 22, 7:13 PM
Unknown Object (File)
Fri, Dec 13, 10:36 AM
Unknown Object (File)
Sat, Nov 23, 2:24 PM
Unknown Object (File)
Sat, Nov 23, 2:24 PM
Unknown Object (File)
Sat, Nov 23, 1:19 PM
Unknown Object (File)
Nov 20 2024, 10:21 AM
Unknown Object (File)
Nov 1 2024, 5:31 PM
Unknown Object (File)
Oct 27 2024, 10:31 AM
Subscribers

Details

Summary

We remove the native-specific uploadSelectedMedia call from updateImageUserAvatar and move it to native-specific nativeUpdateUserImageAvatar.

As of this diff, updateImageUserAvatar now takes (imageAvatarUpdateRequest: ImageAvatarDBContent) rather than the native-specific (selection: NativeMediaSelection).

This diff also eliminates all usages of useUploadSelectedMedia, which means we can remove the prop in a subsequent diff at which point BaseEditUserAvatarProvider will be platform-agnostic! That'll let us consolidate BaseEditUserAvatarProvider, WebEditUserAvatarProvider, and NativeEditUserAvatarProvider.


Depends on D8345

Test Plan

Continue to be able to set image avatars on native as expected.

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul published this revision for review.Jun 27 2023, 6:25 PM
This revision is now accepted and ready to land.Jun 28 2023, 10:11 AM
This revision was landed with ongoing or failed builds.Jul 28 2023, 2:04 PM
This revision was automatically updated to reflect the committed changes.