Handle access controlled assets in ManifestMerger - #684
JackLewis-digirati wants to merge 1 commit into
Conversation
|
Explaining the approach for reviewers: Manifest-level access services ( Protagonist's NamedQuery response carries access services at two levels:
To build the merged manifest's
Only walking the built canvases (not the raw NamedQuery items) is what satisfies "referenced by at least one asset" - it guards against the NQ returning assets that aren't actually in this manifest's Sound/Video fix Separately found and fixed a bug: when cloning painting-annotation bodies (
The auth context ( |
63c9e2f to
316f83e
Compare
What does this change?
Resolves #569
This PR modifies
manifestMergerto correctly copy themanifest.servicesblock when assets have access control services.Additionally, this fixes an issue with audio/video assets not correctly copying service blocks