samvera-labs / ramp

Interactive, IIIF powered audio/video media player React components library. Styleguidist Docs: https://samvera-labs.github.io/ramp/
https://ramp.avalonmediasystem.org/
30 stars 5 forks source link

Change Manifest Parsing Strategy #606

Open joncameron opened 1 month ago

joncameron commented 1 month ago

Description

Moving manifest parsing to a central location so it can be managed with state that doesn't require components to do unnecessary processing.

This can be done before or in parallel with the state management investigation (samvera-labs/ramp#605).

These changes will require thorough testing. Most of it could be handled in unit testing; tests will most likely need to be added as parsing is changed. This should not change functionality, but with changing the parsing strategy it should still be expected that all the fields are still coming to the right place for consumption by Ramp components.

Done Looks Like

QA

elynema commented 1 month ago

Dananji is testing vault and manifesto for parsing manifests. Vault errs when we have inaccessible items with no ID.