Individual overrides when placing multiple holds

[Lindsay Stratton]

Evergreen 3.6.5

Situation:

Staff with CREATE_DUPLICATE_HOLDS and HOLD_ITEM_CHECKED_OUT.override permissions is placing multiple holds on a title for a book group account. The account already has a copy of the requested title checked out.

Each hold being placed has an individual Override button that must be clicked in order to place the holds.

Ideally, the override would only need to be done once to place all holds.

[Dan Briem (dbriem)]

Branch for testing: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dbriem/lp1954681_place_hold_override_all

This adds an override all button above the override column if there are multiple overridable holds.

[Dawn Dale (ddale)]

Maybe I am not understanding the purpose of this request. I do not see an override column. I do see the "override all hold-blocking conditions possible". When placing holds and one is already checked out to patron A, the hold is placed for patron A and all other patrons with the exception of a patron that has a block.

In my opinion, I would not want all conditions to be overridden. Does this override button only override holds when the patron already has a copy checked out?

Thanks,

[Dan Briem (dbriem)]

Dawn, you're referring to the bucket interface for hold groups, correct?

I thought the bug description referred to placing duplicate holds on one account in the Angular staff catalog.

If the "Maximum number of duplicate holds allowed" org unit setting is greater than 1, you have the option to select the number of copies you want when placing a hold. If there is an override condition, the reason is displayed with an override button for each hold.

This pull request adds an override all button so you don't have to click each one. But, maybe I'm off-track completely and misunderstood the bug description.

[Terran McCanna (tmccanna)]

I interpreted it the same way as Dan.

I set the "Maximum number of duplicate holds" to 5 and assigned the CREATE_DUPLICATE_HOLDS and HOLD_ITEM_CHECKED_OUT.override permissions to a staff user. I logged in as that user, checked an item out to a patron, and then placed additional holds for that patron and got the new button as you described and it worked great. I also tried clicking some of the individual overrides and then going back and clicking the override all button and that worked great, too. This is a nice improvement.

My sign off is at:
https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp1954681_place_hold_override_all_signoff

[Michele Morgan (mmorgan)]

This looks great! Pushed to master for release 3.10.

Thanks Dan and Terran!