Skip to content

Adding new v4 validator action #2453

Merged
azaroth42 merged 13 commits intoprezi-4from
add_action
Apr 9, 2026
Merged

Adding new v4 validator action #2453
azaroth42 merged 13 commits intoprezi-4from
add_action

Conversation

@glenrobson
Copy link
Copy Markdown
Member

No description provided.

Using main validator
@glenrobson
Copy link
Copy Markdown
Member Author

Looks like there are some validation errors that need to be fixed in the manifests. Ill sort those in this pull request

@glenrobson glenrobson marked this pull request as draft March 25, 2026 18:10
@glenrobson
Copy link
Copy Markdown
Member Author

Currently failing on the duplicate id check. I need to add body to the list of allowed places for a duplicate id.

But should the first sub-scene be embedded:

https://preview.iiif.io/api/prezi-4/presentation/4.0/example/uc06_scene_in_scene.json

@glenrobson
Copy link
Copy Markdown
Member Author

I think this is ready to go but I would like someone to confirm the changes I've made to scene in scene make sense:

The first sub scene is embedded in full:

"id": "https://iiif.io/api/presentation/4.0/example/uc06/scene/1",

but the second instance of the scene is only a reference:

"id": "https://iiif.io/api/presentation/4.0/example/uc06/scene/1",

Is that right? It would mean a lot of duplication to duplicated the sub scene twice.

@glenrobson
Copy link
Copy Markdown
Member Author

Update example with a range with behaviour sequence

@glenrobson glenrobson marked this pull request as draft April 8, 2026 17:57
Adding uc08_3d_annotation example manifest
Adding uc08_3d_comments_with_cameras.json
@azaroth42 azaroth42 merged commit 43527d9 into prezi-4 Apr 9, 2026
2 of 3 checks passed
@azaroth42 azaroth42 deleted the add_action branch April 9, 2026 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants