Summary: | Phone & SMS transports always displayed in overdue status triggers | ||
---|---|---|---|
Product: | Koha | Reporter: | Kyle M Hall (khall) <kyle> |
Component: | Circulation | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | andrew, caroline.cyr-la-rose, dcook, fridolin.somers, gmcharlt, jonathan.druart, kyle.m.hall, lucas, victor |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
21.05.00,20.11.02
|
Circulation function: | |
Attachments: |
Bug 26953: Hide phone and itiva from "Overdue notice/status triggers"
Bug 26953: Hide phone and itiva from "Overdue notice/status triggers" Bug 26953: Hide phone and itiva from "Overdue notice/status triggers" Bug 26953: Hide phone and itiva from "Overdue notice/status triggers" |
Description
Kyle M Hall (khall)
2020-11-06 13:57:45 UTC
I don't them in the notice editor if the prefs are off. Created attachment 114089 [details] [review] Bug 26953: Hide phone and itiva from "Overdue notice/status triggers" If the prefs are off we should not display the checkboxes Test plan: 1. Turn off PhoneNotification and TalkingTechItivaPhoneNotification 2. Go to the "Overdue notice/status triggers" and confirm that phone and itiva columns are not displayed 3. Turn the prefs on and confirm the checkboxes are now displayed Created attachment 114090 [details] [review] Bug 26953: Hide phone and itiva from "Overdue notice/status triggers" If the prefs are off we should not display the checkboxes Test plan: 1. Turn off PhoneNotification and TalkingTechItivaPhoneNotification 2. Go to the "Overdue notice/status triggers" and confirm that phone and itiva columns are not displayed 3. Turn the prefs on and confirm the checkboxes are now displayed (In reply to Jonathan Druart from comment #1) > I don't them in the notice editor if the prefs are off. I don't *see* them in the notice editor if the prefs are off. > I don't *see* them in the notice editor if the prefs are off.
Confirmed, I do not see them either when the prefs are off
Created attachment 114293 [details] [review] Bug 26953: Hide phone and itiva from "Overdue notice/status triggers" If the prefs are off we should not display the checkboxes Test plan: 1. Turn off PhoneNotification and TalkingTechItivaPhoneNotification 2. Go to the "Overdue notice/status triggers" and confirm that phone and itiva columns are not displayed 3. Turn the prefs on and confirm the checkboxes are now displayed Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Created attachment 114621 [details] [review] Bug 26953: Hide phone and itiva from "Overdue notice/status triggers" If the prefs are off we should not display the checkboxes Test plan: 1. Turn off PhoneNotification and TalkingTechItivaPhoneNotification 2. Go to the "Overdue notice/status triggers" and confirm that phone and itiva columns are not displayed 3. Turn the prefs on and confirm the checkboxes are now displayed Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Pushed to master for 21.05, thanks to everybody involved! Pushed to 20.11.x for 20.11.02 Doesn't apply cleanly to 20.05, please rebase if needed |