Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support IIIF Search 1.0 #705

Open
tomcrane opened this issue Dec 11, 2019 · 4 comments
Open

Support IIIF Search 1.0 #705

tomcrane opened this issue Dec 11, 2019 · 4 comments

Comments

@tomcrane
Copy link
Contributor

tomcrane commented Dec 11, 2019

UV version:
All

I suspect this supersedes #132

@nicolasfranck mentioned in IIIF Slack:

the universalviewer now only supports version 0.9, and neglects version 1.0 despite the fact that is practically the same

I replied:
Yes, unfortunately. The Search functionality at Wellcome is basically 0.9.3 and the server part of that was never updated for 1.0 after it went live, and it's that search functionality that the UV's implementation was designed for. As njfranck says it wouldn't take much to make it 1.0-compliant but that's never been at the top of anyone's backlog. As ever, PRs welcome.

It's probably a good first bug.

@tomcrane
Copy link
Contributor Author

It's probably just a matter of recognising the @context for Search 1.

@nicolasfranck
Copy link
Contributor

apparently already implemented by @edsilv in IIIF-Commons/manifold@6b3b843

@edsilv
Copy link
Member

edsilv commented Dec 11, 2019

Yeah. If/when we merge the webpack PR (having a call with @stephenwf, @demiankatz,and @Geoffsc about this today) we will get the latest manifold.

@LlGC-szw
Copy link

All issues will be triaged for further investigation or closure by the 28 September 2023. If your issue is still relevant and would like for it be investigated further please comment by 14 September 2023.

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

No branches or pull requests

4 participants