-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Cover not showing on merge toolbar or author page #9064
Comments
Hi! I would like to try to work on this task, could I be assigned to this? |
@jjessieyang I assigned you to work on it. Good luck! |
I'm having some trouble setting things up locally. I think I've followed all the guides for setting up docker, production book data, etc. as best I can, but I still can't see any books other than what's provided in the pre-loaded dataset or links to view by Subjects. I've read through issue #1864 but I still can't get it to work locally. How should I go about working on this task? This is my first time contributing to an open source project so any advice is greatly appreciated. Thank you! |
@jjessieyang Let me know if you have further questions. |
Okay thank you! Do you have any suggestions on where I should look to fix this bug? I think I'm also just a bit confused why this book is the only one that has this error. |
@jjessieyang, one approach can be to try to figure out what's different between the working case and the not working case. Here, it looks as if this cover is coming from Internet Archive's cover service rather than from Open Library's. Were I to hazard a guess as to what's going on, I would say that the places that the cover doesn't show up aren't using Internet Archive's cover service as a fall back. If you visit https://openlibrary.org/works/OL33647732W.json and https://openlibrary.org/books/OL45606286M.json you'll see that this book doesn't have a cover, yet curiously one shows up. C.f. https://openlibrary.org/works/OL66554W.json, which does have a cover. If you mouse over the cover image where it does display for the book in this issue, you can see that unlike many covers on Open Library, the link isn't to covers.openlibrary.org/something, but to https://archive.org/download/heraldosdelpacha0000chur/page/cover_w180_h360.jpg. If you do openlibrary/openlibrary/plugins/upstream/models.py Lines 75 to 84 in 69b903a
Note: a better From here you'll want to look into how the 'broken' images are being loaded. As a quick sketch, you might want to right click -> 'inspect' the elements that are broken. For example, if you do that on image in the lower left of the screen where it says "1 edition selected" (i.e. the place with the red arrow pointing to it in the first image in this issue summary), you'll see:
With the help of Hopefully that gets you started. If you get stuck, please ask questions. |
Problem
Why does this cover, which has been there since last year, not show on either of these page spots?
Evidence / Screenshot
Relevant URL(s)
Reproducing the bug
Context
Notes from this Issue's Lead
Proposal & constraints
Related files
Stakeholders
The text was updated successfully, but these errors were encountered: