Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Video Element #81

Open
jsavella opened this issue Aug 6, 2024 · 3 comments
Open

Implement Video Element #81

jsavella opened this issue Aug 6, 2024 · 3 comments

Comments

@jsavella
Copy link
Collaborator

jsavella commented Aug 6, 2024

Original: https://www.esri.com/en-us/about/about-esri/overview
Edge Delivery site: https://main--esri--aemsites.hlx.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
Copy link
Collaborator

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
Copy link
Collaborator

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
Copy link
Collaborator

One note: there may be other parameters after the ID (1_9gsx27x2) those should be retained if they exist.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants