Bug 29456

Summary: "Auto renewal" and "Hold reminder" notice shown as "unknown" on the patron category list view
Product: Koha Reporter: Kelly McElligott <kelly>
Component: System AdministrationAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: fridolin.somers, gmcharlt, jonathan.druart, kyle
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.11.00,21.05.05
Bug Depends on: 18532    
Bug Blocks:    
Attachments: visual
Bug 29456: (bug 18532 follow-up) handle Auto_Renewals messages in category
Bug 29456: (bug 18532 follow-up) handle Auto_Renewals messages in category
Bug 29456: (bug 18532 follow-up) handle Auto_Renewals messages in category
Bug 29456: Add Hold_Reminder description to patron categories overview page

Description Kelly McElligott 2021-11-10 19:41:32 UTC
Created attachment 127518 [details]
visual

When looking at the Patron Category overview, a library gets a glimpse at all the pieces of that patron category. One of the pieces is the summary of notices, currently, the last notice that is listed is called "unknown'- which should be named "auto-renew- see screenshot. This should have the correct text instead of unknown.
Comment 1 Jonathan Druart 2021-11-16 14:00:13 UTC
Created attachment 127692 [details] [review]
Bug 29456: (bug 18532 follow-up) handle Auto_Renewals messages in category

The new message appears in the list on the edit view, but the list view
is showing "unknown"

Test plan:
Set AutoRenewalNotices to "according to patron messaging prefs"
Edit a patron category
Tick all the checkboxes
On the category list view you should see a correct display in the
"Messaging" column
Comment 2 PTFS Europe Sandboxes 2021-11-16 14:12:45 UTC
Created attachment 127693 [details] [review]
Bug 29456: (bug 18532 follow-up) handle Auto_Renewals messages in category

The new message appears in the list on the edit view, but the list view
is showing "unknown"

Test plan:
Set AutoRenewalNotices to "according to patron messaging prefs"
Edit a patron category
Tick all the checkboxes
On the category list view you should see a correct display in the
"Messaging" column

Signed-off-by: Kelly McElligott <kelly@bywatersolutions.com>
Comment 3 Kelly McElligott 2021-11-16 14:13:20 UTC
Works great!  Signing off through sandboxes.
Comment 4 Katrin Fischer 2021-11-18 11:08:27 UTC
Created attachment 127783 [details] [review]
Bug 29456: (bug 18532 follow-up) handle Auto_Renewals messages in category

The new message appears in the list on the edit view, but the list view
is showing "unknown"

Test plan:
Set AutoRenewalNotices to "according to patron messaging prefs"
Edit a patron category
Tick all the checkboxes
On the category list view you should see a correct display in the
"Messaging" column

Signed-off-by: Kelly McElligott <kelly@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Katrin Fischer 2021-11-18 11:08:31 UTC
Created attachment 127784 [details] [review]
Bug 29456: Add Hold_Reminder description to patron categories overview page

This was missed when initially adding the notice.

To test:
- Go to Administraiton > patron categories
- Edit a patron category and check "Hold reminder" in messaging
  preferences, save.
- Go to the overview page and verify it shows as 'Unknown'.
- Apply patch.
- Descrpition should now display.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Katrin Fischer 2021-11-18 11:35:02 UTC
Strictly speaking this is a string patch, but as the string already exists in other spots, I believe we could be lucky here. And an English string showing would also be better than "Unknown".
Comment 7 Jonathan Druart 2021-11-19 14:15:52 UTC
Pushed to master for 21.11, thanks to everybody involved!
Comment 8 Kyle M Hall 2021-11-19 17:00:44 UTC
Pushed to 21.05.x for 21.05.05
Comment 9 Fridolin Somers 2021-11-20 02:09:28 UTC
Depends on Bug 18532 not in 20.11.x