lending: if book is not available to borrow, offer to add it to "my books to borrow" list

Bug #704097 reported by mangtronix
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Open Library
Confirmed
High
Anand Chitipothu

Bug Description

If user tries to borrow a book that is checked out, provide link to add it to a list of books they want to borrow (similar to a "wish list").

Tags: lending lists
mangtronix (mang)
tags: added: lending
tags: added: lists
George (george-archive)
Changed in openlibrary:
assignee: nobody → George (george-archive)
Revision history for this message
George (george-archive) wrote :

Not sure we should create a list for people automatically. I think it's fine to just present the normal "Add to List" UI widget there.

See 2 mocks:
- add to list widget + link to browse again
- if you add to a list, you get an AJAXed confirmation, and new links to the list and browse...

Revision history for this message
George (george-archive) wrote :

Add to List

Revision history for this message
George (george-archive) wrote :

Added to List - confirmation

Changed in openlibrary:
assignee: George (george-archive) → Lance Arthur (lance-arthur)
Revision history for this message
George (george-archive) wrote :

Lance - can you pls mark-up?

Revision history for this message
George (george-archive) wrote :
Revision history for this message
Lance Arthur (lance-arthur) wrote :

I added in the code as-is, but it uses the existing List Widget which was designed to be an all-in-one listing feature for use on the regular pages, so it will always perform the same services (display any lists the book is on, display a header including a link to the lists, etc., then display a post-add of the display with the drop-down intact and the new list listing displayed) so Anand (or someone) will need to construct/alter the existing widget to look/act like the above comps.

The code on the page simply formats the content as required. It's hidden under an Admin-only envelope and if you add the book to a list, it will actually function, but is incapable of displaying the resulting acknowledgment as designed in this bug.

Revision history for this message
George (george-archive) wrote :

Anand - can you please hook this up? We'll need it in place by the end of next week.

Changed in openlibrary:
assignee: Lance Arthur (lance-arthur) → Anand Chitipothu (anandology)
status: New → Confirmed
importance: Undecided → High
Revision history for this message
George (george-archive) wrote :

Anand - Please see if you can hook this up THIS WEEK.

Revision history for this message
George (george-archive) wrote :

Note - it looks like the transaction of adding a checked-out edition to a list is working, it's just the confirmation on the /borrow template that needs to be tied in.

Revision history for this message
George (george-archive) wrote :

Here's an example of the display now: http://openlibrary.org/books/OL6050212M/Simple_cooking_for_the_epicure/borrow

(Though, not sure how long this loan will be outstanding. According to /admin/loans, this PDF loan was checked out 7 hours ago, but is still not downloaded.)

Revision history for this message
mangtronix (mang) wrote :

/admin/loans still isn't updating automatically. You can click the "refresh" button next to a loan to to go to the debugging page which first refreshes the loan status. Keeps getting pushed off.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.