[native] change type for useProcessSelectedMedia from MediaLibrarySelection to NativeMediaSelection
ClosedPublic
Actions

Authored by ginsu on Apr 19 2023, 6:47 PM.

Details

Reviewers

atul
ashoat

Commits

rCOMMeefebccc4b08: [native] change type for useProcessSelectedMedia from MediaLibrarySelection to…

Summary

we need to do this so we can extend useProcessSelectedMedia to handle images from camera capture in addition to images from photo gallery selection

Test Plan

flow

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ginsu created this revision.Apr 19 2023, 6:47 PM

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptApr 19 2023, 6:47 PM

ginsu published this revision for review.Apr 19 2023, 6:47 PM

ginsu added reviewers: atul, ashoat.

ashoat accepted this revision.Apr 19 2023, 6:57 PM

This revision is now accepted and ready to land.Apr 19 2023, 6:57 PM

Harbormaster failed remote builds in B18570: Diff 25444!Apr 19 2023, 7:02 PM

ginsu added a child revision: D7549: [native] introduce naive user avatar camera modal.Apr 19 2023, 7:15 PM

This revision was landed with ongoing or failed builds.Apr 19 2023, 7:17 PM

Closed by commit rCOMMeefebccc4b08: [native] change type for useProcessSelectedMedia from MediaLibrarySelection to… (authored by ginsu). · Explain Why

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMMeefebccc4b08: [native] change type for useProcessSelectedMedia from MediaLibrarySelection to….

Revision Contents
Changeset List

Path

Size

native/

avatars/

avatar-hooks.js

5 lines

Diff 25446

View Options

native/avatars/avatar-hooks.js

[native] change type for useProcessSelectedMedia from MediaLibrarySelection to NativeMediaSelectionClosedPublicActions