wishlist: add new best-hold sort option for proximity between user home library and item owning library

Bug #1869794 reported by Andrea Neiman
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

This development is sponsored by Westchester Library System.

Does what it says - adds an option to the Best Hold Selection Sort Order interface to allow taking into consideration the proximity of item owning library (in a non-float setting) and patron home library.

Setting this option to the top of the BHSSO will allow an institution to prioritize using Library A's copies to fill holds for Library A's patrons.

Branch forthcoming.

Revision history for this message
Jason Etheridge (phasefx) wrote :
tags: added: pullrequest
Revision history for this message
Lindsay Stratton (lstratton) wrote :

Representing Westchester Library System, I tested this and confirm that it works as advertised.

Our org structure is composed of a single consortia with single outlet libraries and libraries with branches.

The "holds chase home lib patrons" prioritizes item lib A=patron lib A holds regardless of request date, or other holds for pickup at lib B, when a lib A item is returned to lib B and a lib A patron has a hold for pickup there.

This BHSSO will also prioritize holds for branch libraries, matching branch A patron holds with lib A items when there is no lib A patron hold.

When there are no lib A or branch A patron holds, a lib A item will fill holds by opportunistic capture or request date.

Andrea Neiman (aneiman)
Changed in evergreen:
milestone: none → 3.5.0
Revision history for this message
Andrea Neiman (aneiman) wrote :

Added a hopeful 3.5.0 target to this - it is a wishlist item but it's small and I'd love to see it in 3.5 if we can get another community signoff (pending RM approval, of course).

Andrea Neiman (aneiman)
description: updated
summary: wishlist: add new best-hold sort option for proximity between user home
- library and item circulating library
+ library and item owning library
Revision history for this message
Bill Erickson (berick) wrote :

Hi, I reviewed (but not tested) the code and have no objections to merging this into 3.5.

Revision history for this message
Ben Shum (bshum) wrote :

Reviewing and testing this code, also sent an email to Lindsey Stratton to request verification on their testing so that we can try adding the additional testing commit lines for this feature prior to merging.

Changed in evergreen:
assignee: nobody → Ben Shum (bshum)
status: New → Triaged
importance: Undecided → Wishlist
Revision history for this message
Lindsay Stratton (lstratton) wrote :

Officially (finally) now:

I have tested this code and consent to signing off on it with my name, Lindsay Stratton, and my email address, <email address hidden>

Revision history for this message
Ben Shum (bshum) wrote :

Added Lindsay's signoff and pushed to master for inclusion in 3.5

Changed in evergreen:
assignee: Ben Shum (bshum) → nobody
status: Triaged → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
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.