To recreate: 1 - Define default messaging preferences for a patron category, checking a transport and the digest only boxes 2 - Create a new patron of a different category 3 - Change the category to the one set above 4 - Note that checkboxes are checked, but digest boxes are disabled 5 - Save patron - the digest boxes are not saved
Created attachment 131062 [details] [review] Bug 30175: Toggle digest boxes when loading messaging preferences To test: 1 - Define default messaging preferences for a patron category, checking a transport and the digest only boxes 2 - Create a new patron of a different category 3 - Change the category to the one set above 4 - Note that checkboxes are checked, but digest boxes are disabled 5 - Save patron - the digest boxes are not saved 6 - Apply patch 7 - Repeat 2&3 8 - Note checkboxes for digest are checked and enabled 9 - Save patron and confirm options are saved
Created attachment 131064 [details] [review] Bug 30175: Toggle digest boxes when loading messaging preferences To test: 1 - Define default messaging preferences for a patron category, checking a transport and the digest only boxes 2 - Create a new patron of a different category 3 - Change the category to the one set above 4 - Note that checkboxes are checked, but digest boxes are disabled 5 - Save patron - the digest boxes are not saved 6 - Apply patch 7 - Repeat 2&3 8 - Note checkboxes for digest are checked and enabled 9 - Save patron and confirm options are saved Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Created attachment 132192 [details] [review] Bug 30175: Toggle digest boxes when loading messaging preferences To test: 1 - Define default messaging preferences for a patron category, checking a transport and the digest only boxes 2 - Create a new patron of a different category 3 - Change the category to the one set above 4 - Note that checkboxes are checked, but digest boxes are disabled 5 - Save patron - the digest boxes are not saved 6 - Apply patch 7 - Repeat 2&3 8 - Note checkboxes for digest are checked and enabled 9 - Save patron and confirm options are saved Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Works as expected, no regressions found PQA
Pushed to master for 22.05, thanks to everybody involved [U+1F984]