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

IIIF Manifest error #127

Open
jaimemears opened this issue May 13, 2021 · 3 comments
Open

IIIF Manifest error #127

jaimemears opened this issue May 13, 2021 · 3 comments
Assignees
Labels
bug Something isn't working question Further information is requested
Projects

Comments

@jaimemears
Copy link
Collaborator

jaimemears commented May 13, 2021

"IIF Manifest Error" and “Error loading the work collection”

We know based on LC developer response that in order to correct these loading issues we'd have to " code to deal with the possibility of a 500 response and catch, retry, and notify the user etc"

We also should handle LOC IIIF manifest itself (not just the Collection reference) failing, and still pass through info from the local manifest.

@jaimemears jaimemears added bug Something isn't working question Further information is requested labels May 13, 2021
@jaimemears jaimemears added this to To do in SA Features via automation May 13, 2021
@jaimemears jaimemears moved this from To do to Needs feedback/action in SA Features May 13, 2021
@adamjarling
Copy link
Owner

So currently if there's an error loading the manifest from LOC's servers, the UI is just relaying that error message to the user. We could do one of 2 things:

  1. Update the text on the error message to make it more user-friendly, or
  2. Suppress the error message entirely, since some of the metadata will still load (from local Manifests)

Let me know what you'd prefer @jaimemears @CourtneyMcClellan ?

@CourtneyMcClellan
Copy link
Collaborator

CourtneyMcClellan commented May 14, 2021 via email

@CourtneyMcClellan CourtneyMcClellan moved this from Needs feedback/action to To do in SA Features May 14, 2021
@adamjarling
Copy link
Owner

@jaimemears If you want to confirm this, I'll hide the errors (in the UI but still let them flow through to the browser console for help in debugging for your devs).

@adamjarling adamjarling moved this from To do to In progress in SA Features May 26, 2021
@jaimemears jaimemears moved this from In progress to Needs feedback/action in SA Features Jun 7, 2021
@jaimemears jaimemears moved this from Needs feedback/action to In progress in SA Features Jun 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
SA Features
In progress
Development

No branches or pull requests

3 participants