It would be great to see, from the circulation rules, smart-rules.pl, what is set in CircControl. The pertinent preferences for holds and return policies appear in the default holds policy section, so it would be nice and increase transparency to see Circ Control setting near the circulation rule matrix and would make trouble-shooting circulation anomalies easier.
+1 - this would be a really nice addition
Created attachment 164439 [details] [review] Bug 36510: Add CircControl and HomeOrHoldingBranch settings to smart-rules To test: 1 - apply patch 2 - restart all 3 - load circ & fines rules, confirm syspref values show in the blue box of rules explanation 4 - change sysprefs and confirm values update on circ rules page
Created attachment 164441 [details] [review] Bug 36510: Add CircControl and HomeOrHoldingBranch settings to smart-rules To test: 1 - apply patch 2 - restart all 3 - load circ & fines rules, confirm syspref values show in the blue box of rules explanation 4 - change sysprefs and confirm values update on circ rules page Signed-off-by: Michelle Spinney <mspinney@clamsnet.org>
Created attachment 164478 [details] [review] Bug 36510: Add CircControl and HomeOrHoldingBranch settings to smart-rules To test: 1 - apply patch 2 - restart all 3 - load circ & fines rules, confirm syspref values show in the blue box of rules explanation 4 - change sysprefs and confirm values update on circ rules page Signed-off-by: Michelle Spinney <mspinney@clamsnet.org> Signed-off-by: hebah <hebah@bywatersolutions.com>
Created attachment 165394 [details] [review] Bug 36510: Add CircControl and HomeOrHoldingBranch settings to smart-rules To test: 1 - apply patch 2 - restart all 3 - load circ & fines rules, confirm syspref values show in the blue box of rules explanation 4 - change sysprefs and confirm values update on circ rules page Signed-off-by: Michelle Spinney <mspinney@clamsnet.org> Signed-off-by: hebah <hebah@bywatersolutions.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Pushed for 24.05! Well done everyone, thank you!
Not backported to 23.11.x