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
F2146160: D8346.id.diff
Sat, Jun 29, 11:25 PM
F2145903: D8346.id29210.diff
Sat, Jun 29, 10:55 PM
F2140608: D8346.diff
Sat, Jun 29, 7:43 AM
Unknown Object (File)
Thu, Jun 27, 9:38 AM
Unknown Object (File)
Tue, Jun 25, 2:12 PM
Unknown Object (File)
Mon, Jun 24, 2:48 PM
Unknown Object (File)
Sun, Jun 23, 10:07 PM
Unknown Object (File)
Sun, Jun 23, 9:03 PM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.