Comment 8 for bug 2000270

Added a check to the holds screen in the staff client that disables the button when an invalid pickup lib is selected. Not completely happy with this since it still allows the user to perform an invalid action (bad UX, grr!) but it should stop the invalid holds from getting placed at least.

Also should modify the org selector to show as invalid when selecting an invalid pickup lib.

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/smayo/lp2000270-Staff_Catalog_Allows_Invalid_Hold_Pickup_Locations

Will still use pullrequest tag because the behavior is better than before but I'll want to revisit this once the the keyboard interaction workarounds are made public. Perhaps as a different bug?