feat(estense): add Jarvis provider and discovery wording cleanup#160
Merged
feat(estense): add Jarvis provider and discovery wording cleanup#160
Conversation
Biblioteca Estense Universitaria in Modena is served by the Jarvis backend at jarvis.edl.beniculturali.it and exposes real IIIF — Presentation v2 and v3, Image API level 2 with tile and zoom. Wire it into Discovery as a first-class provider so the downloader, Studio reader, and Settings pane work without the on-the-fly conversions needed for ICCU. - resolvers/estense.py: recognise manifest URLs (v2 and v3), Mirador wrapper URLs on the Jarvis host, and bare UUIDs (8-4-4-4-12). Helpers build_manifest_url / build_viewer_url keep URL construction in one place. - resolvers/search/estense.py: use the Spring HATEOAS endpoint findBySgttOrAutnOrPressmark (short title / author / pressmark in one call) with Spring Pageable (0-based). Parse _embedded.culturalItems and expose totalElements / totalPages through the shared _search_total_results / _search_total_pages raw meta used by the Discovery UI. - providers.py: register "Estense" with search_strategy="estense", search_mode="search_first", sort_order=75 (between e-codices and Harvard). - network_policy.py: new "estense" key plus aliases (estense, edl, biblioteca estense, etc.) with the default non-custom policy — the Jarvis backend is a modern service and behaves like the IIIF-native providers, so no custom rate limiting is needed. - routes/discovery_handlers.py: add "estense" to _PAGINATABLE_STRATEGIES so Carica altri works. - components/settings/panes/network.py: dedicated Network & Libraries card for Estense matching the existing pattern. - tests/test_estense_unit.py + fixture: 11 unit tests covering UUID extraction across input shapes, URL builders, resolver behavior, search parsing against a real Jarvis response and Spring Pageable mapping. - Docs (provider-support, configuration, CONFIG_REFERENCE, discovery guide): Estense listed in the registry, added to the matrix and to the paginable provider list, dedicated Per-Provider Notes section with accepted inputs and SPA URL caveats. Refs #105
nikazzio
changed the title
nikazzio
added
area:discovery
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Related
Closes #105
Verification