Page MenuHomePhabricator

[native] Display thumbnail in `Multimedia` component for videos
ClosedPublic

Authored by atul on Sep 22 2022, 8:58 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 21, 11:58 AM
Unknown Object (File)
Sun, Apr 21, 11:57 AM
Unknown Object (File)
Sun, Apr 21, 11:57 AM
Unknown Object (File)
Sun, Apr 21, 11:57 AM
Unknown Object (File)
Sun, Apr 21, 11:53 AM
Unknown Object (File)
Mar 26 2024, 9:42 PM
Unknown Object (File)
Mar 13 2024, 8:20 PM
Unknown Object (File)
Mar 10 2024, 8:59 AM
Subscribers

Details

Summary

Now that we have the persistence and retrieval of video thumbnails figured out, we need to actually display them in the native app.

In this diff we just retrieve the thumbnailURI instead of uri from mediaInfo for media of type video.

This enables a minimally viable end-to-end video experience.

NOTE: Still need to take a closer look at the reportURIDisplayed(...) logic and understand what's going on there better before landing this diff.
Test Plan

See video:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Sep 22 2022, 9:04 AM
Harbormaster failed remote builds in B12388: Diff 17004!
atul requested review of this revision.Sep 22 2022, 9:09 AM
atul planned changes to this revision.Sep 22 2022, 9:26 AM
atul requested review of this revision.Sep 22 2022, 9:50 AM
atul requested review of this revision.Sep 22 2022, 4:43 PM
This revision is now accepted and ready to land.Sep 23 2022, 8:29 AM