I think in current versions of Koha the RoutingSerials is used to indicate if routing lists are used. Routing related functionality is hidden from screen according to this system preference. The description doesn't reflect that well: Add/Don't add received serials to the routing list.
Could we make the description... Manage/Don't manage routing lists in the serials module. ?
Created attachment 34023 [details] [review] Bug 9958: Improve RoutingSerials sys pref description The RoutingSerials preference controls if the routing list related functionality is visible in the serials module. To test: - Go to the system preference editor - RoutingSerials - Check the current description - Apply patch - Check the new description - Verify functionality matches description
Making a suggestion for a new wording :)
Created attachment 34445 [details] [review] Bug 9958: Improve RoutingSerials sys pref description The RoutingSerials preference controls if the routing list related functionality is visible in the serials module. To test: - Go to the system preference editor - RoutingSerials - Check the current description - Apply patch - Check the new description - Verify functionality matches description NOTE: The preference change is merely cosmetic. The value is still YesNo. A few git reset --hard origin/master and page refreshes confirmed expected values. Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Native English speaker needed :) Kyle maybe?
Created attachment 34548 [details] [review] [PASSED QA] Bug 9958: Improve RoutingSerials sys pref description The RoutingSerials preference controls if the routing list related functionality is visible in the serials module. To test: - Go to the system preference editor - RoutingSerials - Check the current description - Apply patch - Check the new description - Verify functionality matches description NOTE: The preference change is merely cosmetic. The value is still YesNo. A few git reset --hard origin/master and page refreshes confirmed expected values. Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Patch pushed to master. Thanks Katrina!
Pushed to 3.18.x will be in 3.18.3
Pushed to 3.16.x, patch will be in 3.16.7
(In reply to Mason James from comment #9) > Pushed to 3.16.x, patch will be in 3.16.7 oops, correction! Contains string changes, holding for 3.16.8
Pushed to 3.16.x, will be in 3.16.7