internetarchive / internetarchive/openlibrary

Error message at Already Read Stats page does not work on merged works

Open
#7,028 0 comments 0 reactions 0 assignees View on GitHub
Lead: @cdrini Priority: 3 Theme: Reading Stats
Dominant language
Python
Stars
6.7k
Forks
2k
Avg merge
2d 19h
Merged PRs (30d)
138

Description

I got an error message telling me that "3 works excluded due to missing data" from the "Most Read Authors" section. I assume it should tell me which books are missing (because that is what it does with the other stats), but in this case, I only get question marks:

(????) by
(????) by
(????) by

Upon looking into it, I saw that three of the books from my list had been merged and remained only as placeholders. Once I removed the books from the list and added them anew, the stats showed up correctly.

I doubt that this behaviour is intended, but it is mostly a cosmetic problem.

### Relevant url?
https://openlibrary.org/people/USERNAME/books/already-read/stats

### Steps to Reproduce
Add a book to your read list
Merge that book

* Actual: There is simply a placeholder without any further info. (????) by

* Expected: The book on the list is replaced by the book it was merged into.

### Details
- **Logged in (Y/N)?**
Yes

- **Browser type/version?**
Firefox 104.0.2

- **Operating system?**
Suse Tumbleweed

- **Environment (prod/dev/local)?**
prod

### Proposal & Constraints
I do not know how this problem could be technically solved, but at the moment, one can either clean the database or constantly render list entries unusable.

### Stakeholders
Almost anyone who uses any list.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.