-
Notifications
You must be signed in to change notification settings - Fork 13
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
Video manifest #108
Comments
Post conference:
Currently it picks the original file with MPEG4 format (as this is what is in both fixtures) for what goes on the canvas. We should look at how we handle this, as maybe the derivative MPEG4 labeled with ".ia.mp4" is more standard regardless of the source format? |
I've been doing some testing with a basic video manifest. Seems to work just fine in UV, Ramp, and Aviary: The same manifest doesn't play in either Mirador or Clover; the console for both seems to refer to CORS issues, but I'm not sure how that's possible if the players above are able to begin playback on the video item without issue? CC @saracarl @benwbrum in case this overlaps with your own testing |
I wonder if @mathewjordan could let us know if we need to set CORS headers on AV material. Are there different JS methods for working with AV, some that require CORS and some that don't? |
I'll take a look to see if I can figure out why this is the case with this Manifest and its resource. The big differentiating factor might be that Mirador and Clover handroll their video element while Aviary, Ramp, (video.js) and UV (mediaelement.js) utilize some helper libraries to assist -- so I'll start there and see what I can dig up. |
@glenrobson to test and consider moving to production if it's still a problem. |
Glen to test if this is still an issue with the live service . |
It looks like everything works but Clover now. Clover still seems to say there is a CORS error. I'm closing this issue and opening on on the Clover Repo but can create a new issue on the IA repo if it is something on their end: |
Comments on current iteration in:
#107
Single video gist: https://gist.github.com/digitaldogsbody/f334ccda5369cf0dc4c2c8cbfa104cbf
Multiple video gist: https://gist.github.com/digitaldogsbody/ea73b6d621f22bc88b8b353a34e9337f
The text was updated successfully, but these errors were encountered: