internetarchive
Remove `/admin/waitinglists` page
Problem
https://openlibrary.org/admin/waitinglists frequently times out because there's too many records and too many queries per record.
The page is presumably supposed to return a list of all books with waitinglists.
Check with @seabelis if this page is needed as I know we have a separate dashboard for Archive.org
One workaround is limiting the number of books returned, which has been tested by #9059 and works on testing
Evidence / Screenshot
Relevant URL(s)
Reproducing the bug
- Go to ...
- Do ...
- Expected behavior:
- Actual behavior:
Context
- Browser (Chrome, Safari, Firefox, etc):
- OS (Windows, Mac, etc):
- Logged in (Y/N): Y
- Environment (prod, dev, local): prod
Notes from this Issue's Lead
Proposal & constraints
Related files
Stakeholders
I've only ever used the one on archive. I'm not sure I knew about this one.
I think we may be able to remove this admin endpoint entirely as the source of truth for checking waitlists is now the archive.org/services/loans API
@pidgezero-one, go for it! Let me know if you have any questions about anything.
Thanks @jimchamp !
My only question so far: does this ticket strictly remove admin/waitinglists (and associated code used only for that page), or does it also need to remove all waiting list code entirely (such as the WaitingLoan class)?
@pidgezero-one, the scope of this issue is limited to removing the admin/waitinglist page and the code associated with it.
Thank you for the clarification @jimchamp ! My PR only covers that scope.