search

Esri / esri-eds

Apache License 2.0
2 stars 0 forks source link

Implement Video Element #231

Open alexcarol opened 1 month ago

alexcarol commented 1 month ago

Migrated from: https://github.com/aemsites/esri/issues/81

Original: https://www.esri.com/en-us/about/about-esri/overview Edge Delivery site: https://main--esri-eds--esri.aem.page//en-us/about/about-esri/overview

Video URL: https://www.esri.com/content/dam/esrisites/en-us/about/about/about-2022/updates-10/about-overview-banner.mp4

Render the following video tag from video url. 

<video muted>
    <source src="https://www.esri.com/content/dam/esrisites/en-us/about/about/about-2022/updates-10/about-overview-banner.mp4 " type="video/mp4">
</video>
twhite313 commented 3 days ago

[Carry over notes from old repo]

We need to account for a couple of patterns: Kaltura and the DAM.

The DAM primarily gets used for motion graphics (banner backgrounds, Story Teller animations) Kaltura is used for lightbox videos -- full length content.

For Kaltura, mediaspace.esri.com, authoring should use the video url, ex: https://mediaspace.esri.com/media/t/1_9gsx27x2 on the backend, URL should be rewritten.

String https://mediaspace.esri.com/media/t should be changed to https://mediaspace.esri.com/embed/secure/iframe/entryId/

@alexcarol Collaborator alexcarol commented on Aug 9 As discussed on the last call, we'll rewrite the urls on the client to allow authors to use urls in the format "https://mediaspace.esri.com/media/t/1_9gsx27x2".

@twhite313 Collaborator twhite313 commented on Aug 9 One note: there may be other parameters after the ID (1_9gsx27x2) those should be retained if they exist.