[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens changed: What|Removed |Added Blocks||30175 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30175 [Bug 30175] Digest options not enabled when populating messaging preferences for a selected category during patron entry -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens changed: What|Removed |Added Blocks||29227 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29227 [Bug 29227] Patron messaging preferences digest show as editable but are not -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #26 from Andrew Fuerste-Henry --- Doesn't apply cleanly to 20.05.x, please rebase if needed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Fridolin Somers changed: What|Removed |Added Version(s)|21.05.00|21.05.00,20.11.06 released in|| CC||fridolin.som...@biblibre.co ||m Status|Pushed to master|Pushed to stable --- Comment #25 from Fridolin Somers --- Pushed to 20.11.x for 20.11.06 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #24 from Jonathan Druart --- Pushed to master for 21.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Jonathan Druart changed: What|Removed |Added Version(s)||21.05.00 released in|| Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Jonathan Druart changed: What|Removed |Added Attachment #120760|0 |1 is obsolete|| --- Comment #23 from Jonathan Druart --- Created attachment 120838 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120838=edit Bug 13613: Move similar JS code to a function -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #22 from Katrin Fischer --- I see this more as a "helper" kind of thing. Nothing bad is going to happen if you check digest and no transport afaik - it just doesn't make sense? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens changed: What|Removed |Added Status|In Discussion |Passed QA --- Comment #21 from Nick Clemens --- (In reply to Jonathan Druart from comment #20) > It's not clear to me how strong we want to prevent that. > > 1. existing values: > a. should we update the DB? > b. what if the form is loaded with all checked? > 2. Shouldn't it be forbidden at lower level? TLDR - I think these are all correct, but this bug is very old and offers improvement - we should file new bugs for remaining issues 1 - a - The current behaviour if only digest selected is to send nothing - we check if they want a digest, then loop the transports - we could clean up, but loading a patron after this bug will fix things and behaviour won't change b - All checked is fine - digestible notices will be digested, and others will not be 2 - Yes, but things are handled in C4/Form/MessagingsPreferences.pm - this needs tests/moved to Koha -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Jonathan Druart changed: What|Removed |Added Status|Passed QA |In Discussion CC||jonathan.druart+koha@gmail. ||com --- Comment #20 from Jonathan Druart --- It's not clear to me how strong we want to prevent that. 1. existing values: a. should we update the DB? b. what if the form is loaded with all checked? 2. Shouldn't it be forbidden at lower level? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #19 from Jonathan Druart --- Created attachment 120760 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120760=edit Bug 13613: Move similar JS code to a function -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #18 from Katrin Fischer --- Thx for putting in the tooltip! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Katrin Fischer changed: What|Removed |Added Attachment #120081|0 |1 is obsolete|| --- Comment #17 from Katrin Fischer --- Created attachment 120740 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120740=edit Bug 13613: (follow-up) Add tooltip when disabled This patch adds a tooltip to the digest checkboxes when disabled. When enabled the tooltip is hidden Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Katrin Fischer changed: What|Removed |Added Attachment #119333|0 |1 is obsolete|| --- Comment #16 from Katrin Fischer --- Created attachment 120739 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120739=edit Bug 13613: (follow-up) Use more unique classes This resolves the issue of being unable to save email as it was being hit by the validator Signed-off-by: Owen Leonard Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Katrin Fischer changed: What|Removed |Added Attachment #119332|0 |1 is obsolete|| --- Comment #15 from Katrin Fischer --- Created attachment 120738 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120738=edit Bug 13613: (Bug 22744 follow-up) Remove js handling for 'none' There was a function and a set of onclick events to handle the 'Do not notify' checkboxes. Those have been removed, so should this code To test: 1 - Sign in to opac 2 - Click on 'your messaging' tab from 'your account' 3 - Check/uncheck some boxes 4 - Note error in the console: Uncaught TypeError: document.opacmessaging.none2 is undefined 5 - Apply patch 6 - repeat 7 - no more error Signed-off-by: Owen Leonard Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Katrin Fischer changed: What|Removed |Added Attachment #119331|0 |1 is obsolete|| --- Comment #14 from Katrin Fischer --- Created attachment 120737 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120737=edit Bug 13613: Don't allow digest to be selected without a digest-able transport selected By default, only the email transport is enabled. This gives the messaging preferences the look of having email and digest and two options. I.E. to some users it appears that you check email for single emails, *or* you check digest for a digest email. To help remove this possible confusion, the digest mode checkbox should be disabled if no digest-able transports are checked for a given notice. Test Plan: 1) Apply this patch 2) For both the staff interface and the opac, note that the digest checkbox is disabled any time that no digest enabled transport ( email, sms if enabled ) is checked for the patron editor. Signed-off-by: Owen Leonard Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens changed: What|Removed |Added Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #13 from Nick Clemens --- Created attachment 120081 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=120081=edit Bug 13613: (follow-up) Add tooltip when disabled This patch adds a tooltip to the digest checkboxes when disabled. When enabled the tooltip is hidden -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #12 from Katrin Fischer --- I like the idea of this, but I feel it would work better with a tool tip and if the checkboxes were greyed out more visible while being unusable. What do you think? We have an example for exactly the tooltip at least on the patron attributes administration page. To test: Add the same attribute with same value to 2 patrons and try to make it unique. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #11 from Owen Leonard --- Created attachment 119333 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119333=edit Bug 13613: (follow-up) Use more unique classes This resolves the issue of being unable to save email as it was being hit by the validator Signed-off-by: Owen Leonard -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #10 from Owen Leonard --- Created attachment 119332 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119332=edit Bug 13613: (Bug 22744 follow-up) Remove js handling for 'none' There was a function and a set of onclick events to handle the 'Do not notify' checkboxes. Those have been removed, so should this code To test: 1 - Sign in to opac 2 - Click on 'your messaging' tab from 'your account' 3 - Check/uncheck some boxes 4 - Note error in the console: Uncaught TypeError: document.opacmessaging.none2 is undefined 5 - Apply patch 6 - repeat 7 - no more error Signed-off-by: Owen Leonard -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Owen Leonard changed: What|Removed |Added Attachment #118379|0 |1 is obsolete|| Attachment #118380|0 |1 is obsolete|| Attachment #118742|0 |1 is obsolete|| --- Comment #9 from Owen Leonard --- Created attachment 119331 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=119331=edit Bug 13613: Don't allow digest to be selected without a digest-able transport selected By default, only the email transport is enabled. This gives the messaging preferences the look of having email and digest and two options. I.E. to some users it appears that you check email for single emails, *or* you check digest for a digest email. To help remove this possible confusion, the digest mode checkbox should be disabled if no digest-able transports are checked for a given notice. Test Plan: 1) Apply this patch 2) For both the staff interface and the opac, note that the digest checkbox is disabled any time that no digest enabled transport ( email, sms if enabled ) is checked for the patron editor. Signed-off-by: Owen Leonard -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Owen Leonard changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens changed: What|Removed |Added Attachment #118648|0 |1 is obsolete|| --- Comment #8 from Nick Clemens --- Created attachment 118742 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118742=edit Bug 13613: (follow-up) Use more unique classes This resolves the issue of being unable to save email as it was being hit by the validator -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens changed: What|Removed |Added Attachment #118381|0 |1 is obsolete|| --- Comment #7 from Nick Clemens --- Created attachment 118648 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118648=edit Bug 13613: (follow-up) Use more unique classes This resolves the issue of being unable to save email as it was being hit by the validator -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Andrew Fuerste-Henry changed: What|Removed |Added Status|Needs Signoff |Failed QA CC||and...@bywatersolutions.com --- Comment #6 from Andrew Fuerste-Henry --- I get the following error when trying to save a patron category with default messaging prefs of SMS, Email, Digest only for Item Due: C4::Members::Messaging::SetMessagingPreference(): DBI Exception: DBD::mysql::st execute failed: Cannot add or update a child row: a foreign key constraint fails (`koha_kohadev`.`borrower_message_transport_preferences`, CONSTRAINT `borrower_message_transport_preferences_ibfk_2` FOREIGN KEY (`message_transport_type`) REFERENCES `message_transport_types` ) [for Statement "INSERT INTO borrower_message_transport_preferences (borrower_message_preference_id, message_transport_type) VALUES (?, ?) " with ParamValues: 0='1', 1="pmp_sms"] at /kohadevbox/koha/C4/Form/MessagingPreferences.pm line 99 at /usr/share/perl5/DBIx/Class/Exception.pm line 77 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #5 from Nick Clemens --- Created attachment 118381 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118381=edit Bug 13613: (follow-up) Use more unique classes This resolves the issue of being unable to save email as it was being hit by the validator -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #4 from Nick Clemens --- Created attachment 118380 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118380=edit Bug 13613: (Bug 22744 follow-up) Remove js handling for 'none' There was a function and a set of onclick events to handle the 'Do not notify' checkboxes. Those have been removed, so should this code To test: 1 - Sign in to opac 2 - Click on 'your messaging' tab from 'your account' 3 - Check/uncheck some boxes 4 - Note error in the console: Uncaught TypeError: document.opacmessaging.none2 is undefined 5 - Apply patch 6 - repeat 7 - no more error -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens changed: What|Removed |Added Attachment #35471|0 |1 is obsolete|| --- Comment #3 from Nick Clemens --- Created attachment 118379 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=118379=edit Bug 13613 - Don't allow digest to be selected without a digest-able transport selected By default, only the email transport is enabled. This gives the messaging preferences the look of having email and digest and two options. I.E. to some users it appears that you check email for single emails, *or* you check digest for a digest email. To help remove this possible confusion, the digest mode checkbox should be disabled if no digest-able transports are checked for a given notice. Test Plan: 1) Apply this patch 2) For both the staff interface and the opac, note that the digest checkbox is disabled any time that no digest enabled transport ( email, sms if enabled ) is checked for the patron editor. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Nick Clemens n...@quecheelibrary.org changed: What|Removed |Added CC||n...@quecheelibrary.org Status|Needs Signoff |Failed QA --- Comment #2 from Nick Clemens n...@quecheelibrary.org --- I tried testing, in terms of not letting me check digest box until I checked email it worked. But I ran into some issues: 1 - When initially trying to click on the digest box (with nothing else checked) my browser focus jumps to the top of the page, same when I click on email box. Third try stayed and let me click email box. 2 - If I had only email enabled I was unable to save any preferences, I got a Please enter a valid email address note in red next to the checkbox. Entering a valid email did not clear it, entering secondary etc. emails did not. Editing the account to add a valid email, saving, then trying to check a messaging box also failed. The only way I could past this error was to enable additional transports, phone or SMS in any combination removed the error amd actually let me check email whether or not I had a valid email address on file. When I switch to master I cannot get this error 3 - While I was prevented from checking digest until another transport was selected, digest wasn't cleared when I unchecked the emial box. It didn't save, which is good, but the box should be cleared too. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Severity|enhancement |normal Assignee|koha-b...@lists.koha-commun |k...@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Patch complexity|Trivial patch |Small patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 Kyle M Hall k...@bywatersolutions.com changed: What|Removed |Added Status|NEW |Needs Signoff Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 13613] Don't allow digest to be selected without a digest-able transport selected
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13613 --- Comment #1 from Kyle M Hall k...@bywatersolutions.com --- Created attachment 35471 -- http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=35471action=edit Bug 13613 - Don't allow digest to be selected without a digest-able transport selected By default, only the email transport is enabled. This gives the messaging preferences the look of having email and digest and two options. I.E. to some users it appears that you check email for single emails, *or* you check digest for a digest email. To help remove this possible confusion, the digest mode checkbox should be disabled if no digest-able transports are checked for a given notice. Test Plan: 1) Apply this patch 2) For both the staff interface and the opac, note that the digest checkbox is disabled any time that no digest enabled transport ( email, sms if enabled ) is checked for the patron editor. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
