Skip to content

Collection improvements v2#51

Merged
stephenwf merged 8 commits into
mainfrom
feature/collection-improvements-v2
Jun 7, 2025
Merged

Collection improvements v2#51
stephenwf merged 8 commits into
mainfrom
feature/collection-improvements-v2

Conversation

@stephenwf

Copy link
Copy Markdown
Member
  • Fixes bug with missing navPlace when serialising
  • Default "provider" label is now {"none": [""]} instead of {"none": ["Unknown"]}
  • Fixed @context value for Manifests/Collections with navPlace extension
  • Added non-standard first, next, prev and total properties to Collections when upgrading to allow tools to use upgraded Collections in code - adding since its confirmed these properties are returning in v4.0. (I've called this "3.1" in the code).
  • Updated cookbook recipe tests, all passing.

@codesandbox-ci

codesandbox-ci Bot commented Jun 7, 2025

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@stephenwf stephenwf merged commit 886a338 into main Jun 7, 2025
3 checks passed
@mathewjordan

Copy link
Copy Markdown
Member

@stephenwf Thank you for bringing in the collections page support early. This will help a lot!

@stephenwf

Copy link
Copy Markdown
Member Author

No problem! I thought I'd get it done while the conference was still fresh!

I'm working on an extension of Vault to support loading too.
IIIF-Commons/iiif-helpers#25

Which has brought out a few details. Such as this:
https://iiif.harvardartmuseums.org/collections/top

When you load a page the ID is the same for every page (it doesn't include ?page= in the ID). Just in case you run into the same issue.

Also it's really tricky to get a "page size" and not all collections out there have a "total" field, so pagination is mostly limited to a "next" if you're loading hosted collections.

Also a build of Theseus here too trying to shake out any potential issues.
https://feature-paginated-collection.theseus-viewer.pages.dev/?iiif-content=https%3A%2F%2Fapi.digitale-sammlungen.de%2Fiiif%2Fpresentation%2Fv2%2Fcollection%2Ftop&collection=https%3A%2F%2Fapi.digitale-sammlungen.de%2Fiiif%2Fpresentation%2Fv2%2Fcollection%2Ftop

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