embedded iframe for videos

[summer-cook]

Story

SoftServ will customize the ability for YouTube videos to appear embedded on a work show page. Videos will not be played back in Universal Viewer.

Related

• 9

Expected Behavior Before Changes

• all works that should show a UV show them
• users would only see either the uv viewer or default image

Expected Behavior After Changes

• If a work has a valid video_embed link saved in its metadata, the work show page will show that video in an iframe.
• If a work does not have a valid video_embed link saved in its metadata, but should show the universal viewer because it has files, it shows the viewer.
• If a work does not have either video_embed metadata or files to show in the viewer, it shows the default image.

Screenshots / Video

Working youtube player

Working vimeo player

Form with error message for invalid url entry

Acceptance

• [ ] Users should be able to add a video embed link from the work-edit page in the dashboard
• [ ] Users can see help text explaining the format for the links, as well as more in depth directions for how to find the link in the project readme
• [ ] The field should support embed links for either youtube or vimeo links
• [ ] The field should not support urls that are not valid youtube or vimeo links in the formats provided in the help text.
• [ ] On the work show page, the user should be able to see the embedded video if they add a link
• [ ] If there is no link added, the work show page will show the UV or the default image (behavior would be the same and prior to this implementation)
• [ ] All 3 show page (cultural, scholarly, default) themes should support the video-embed viewer

Note

• ability to import the video-embed links with bulkrax will be done in the bulkrax configuration ticket
• this still needs to be added to the 2 other worktypes which are in flight

[summer-cook]

@labradford

My comments are just suggestions, I think it's ok to approve. I do think the conditional rendering of the UV/Video/Representative media is a bit convoluted.

Great job remembering the themes!

I know, originally i wanted to just put it in the representative_media partial and be done with it, but then that ALSO required changes to the show page since there are viewer related conditionals in there too. I will test out your suggestions though, thank you!

[summer-cook]

@labradford i just pushed up the show page views again with your suggestions, but had to do some weird things with the default one to get the columns to work. let me know if you want to take a look again