Provide captions support on A/V items #393
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
swandog30
approved these changes
Aug 14, 2024
This updates UV from v3 to v4, and rather than bundling it with Webpack, we're pulling it in from a CDN. This reduces our bundle size, and offloads the serving of this particular asset to the CDN.
This puts a rendering item into the IIIF manifest for non-playlist A/V items, and adds an endpoint for fetching them
Unsure if changes to the main css file are being picked up for deployment
This is no longer useful with UV v4 because config isn't loaded from and endpoint
DanOlson
force-pushed
the
captions-support
branch
2 times, most recently
from
abb02aa
to
8e1ee7b
Compare
Previously recorded cassettes were written assuming a specifc KALTURA_SESSION environment variable was present. That variable is used to generate the kalsig param as well (a request signature), and so we need to account for both the missing env var, and the effect that has on the signature.
DanOlson
force-pushed
the
captions-support
branch
from
8e1ee7b
to
0234522
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This provides both the backend support necessary to format the IIIF manifest with caption data as well as a forked version of UniversalViewer (loaded from a CDN) that can interpret captions from the manifest. Captions are ingested from ContentDM, stored in Solr (but not indexed), and are excluded by default in the majority of requests the app makes to Solr. A dedicated endpoint,
/tracks/:item_id/entry/:entry_idsurfaces the caption data in VTT format. UV will generate HTML<track>elements for each available track, using that endpoint as thesrcattribute, and will render the track content on top of the viewer.The CDN is jsDelivr, and is currently sourcing the distribution files from my https://github.com/DanOlson/uv-dist/ repo. This is just the output of building my fork of the UV app locally, which I also have NPM linked to my fork of IIIF-Commons/iiif-av-component, https://github.com/DanOlson/iiif-av-component in order to provide caption support for both single A/V items and playlists (manifests with ranges).
Pull requests for UV and iiif-av-component updates are here:
UV: UniversalViewer/universalviewer#1047
iiif-av-component: IIIF-Commons/iiif-av-component#95
Upon merging, we will need to run the ETL for all A/V items in the catalog, and as that progresses, caption support will gradually roll out.