Bug 31354 - Clarify options for 'Change messaging preferences to default for this category?' popup
Summary: Clarify options for 'Change messaging preferences to default for this categor...
Status: Failed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial
Assignee: Sam Lau
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-08-12 15:49 UTC by Sara Brown
Modified: 2024-11-15 08:32 UTC (History)
7 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 31354: Use modal popup for messaging preferences when editing a patron's category. (7.13 KB, patch)
2024-07-03 22:02 UTC, Sam Lau
Details | Diff | Splinter Review
Bug 31354: Keep same functionality as before (1.14 KB, patch)
2024-07-03 22:17 UTC, Sam Lau
Details | Diff | Splinter Review
Bug 31354: Use modal popup for messaging preferences when editing a patron's category. (7.13 KB, patch)
2024-07-03 22:29 UTC, Sam Lau
Details | Diff | Splinter Review
Bug 31354: Use modal popup for messaging preferences when editing a patron's category. (7.18 KB, patch)
2024-07-07 06:56 UTC, David Nind
Details | Diff | Splinter Review
Something looks a bit weird here (33.51 KB, image/png)
2024-09-13 15:15 UTC, Paul Derscheid
Details
Bug 31354: Adjust modal for Bootstrap5 changes (2.17 KB, patch)
2024-09-23 22:56 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Sara Brown 2022-08-12 15:49:33 UTC
The options for the 'Change messaging preferences to default for this category?' popup that appears when editing a patron's category are Cancel and OK, but I think No instead of Cancel and Yes instead of OK would be clearer. A library could reasonably think that Cancel refers to cancelling the change in category, and No would make it clearer that refers to the preferences only.
Comment 1 Lucas Gass (lukeg) 2022-08-15 17:30:02 UTC
To make this language more clear I think we should use a modal popup instead of a JS alert. 

+1
Comment 2 Sam Lau 2024-07-03 22:02:57 UTC Comment hidden (obsolete)
Comment 3 Sam Lau 2024-07-03 22:17:39 UTC Comment hidden (obsolete)
Comment 4 Sam Lau 2024-07-03 22:29:49 UTC
Created attachment 168484 [details] [review]
Bug 31354: Use modal popup for messaging preferences when editing a patron's category.

To test:
1) In Administration->Patron categories, edit the 'Home Bound' category and select every messaging preferences.
2) Edit the 'Patron' category and select email for the 'Item due' messaging pref.
3) Visit a patrons page from any category except Homebound. Keep note of their messaging preferences (will probably just be none)
4) Edit this patron and change their category to 'Homebound'. You should be greeted with a modal that asks if you want to "Change messaging preferences to default for this category?". Select yes. Save changes on the edit form
5) In the details page, note the messaging preferences were set to the defualt for the Homebound category (all selected)
6) Edit patron again, this time change their category to 'Patron'.
7) When the modal pops up, select no. Save the edit form.
8) In details page, the messaging prefs should not have changed.
Comment 5 David Nind 2024-07-07 06:56:59 UTC
Created attachment 168578 [details] [review]
Bug 31354: Use modal popup for messaging preferences when editing a patron's category.

To test:
1) In Administration->Patron categories, edit the 'Home Bound' category and select every messaging preferences.
2) Edit the 'Patron' category and select email for the 'Item due' messaging pref.
3) Visit a patrons page from any category except Homebound. Keep note of their messaging preferences (will probably just be none)
4) Edit this patron and change their category to 'Homebound'. You should be greeted with a modal that asks if you want to "Change messaging preferences to default for this category?". Select yes. Save changes on the edit form
5) In the details page, note the messaging preferences were set to the defualt for the Homebound category (all selected)
6) Edit patron again, this time change their category to 'Patron'.
7) When the modal pops up, select no. Save the edit form.
8) In details page, the messaging prefs should not have changed.

Signed-off-by: David Nind <david@davidnind.com>
Comment 6 David Nind 2024-07-07 09:09:02 UTC
Not related to this bug, but I identified this issue - Bug 37269 - Saving edits to a patron - error message clears or resets some fields.
Comment 7 Paul Derscheid 2024-09-13 15:15:05 UTC
Created attachment 171474 [details]
Something looks a bit weird here
Comment 8 Paul Derscheid 2024-09-13 15:19:54 UTC
Thanks for the effort to make this a real modal!

The close button is however unstyled and I run into the following issues:
1) After running through the test plan I changed the patron category back to Homebound again
2) I then selected 'yes' in the modal.
3) Then I selected Homebound again.
4) Then I tried to select 'no' in the modal, but there was no reaction.
5) There was an alert with a spinner above the messaging preferences, though. Informing me on that message preference details were loaded.

Could you try to reproduce this issue and then set it to Failed QA, or supply a patch for it?
Comment 9 Lucas Gass (lukeg) 2024-09-23 22:56:15 UTC
Created attachment 171905 [details] [review]
Bug 31354: Adjust modal for Bootstrap5 changes
Comment 10 Lucas Gass (lukeg) 2024-09-23 22:56:47 UTC
> Could you try to reproduce this issue and then set it to Failed QA, or
> supply a patch for it?

Paul can you try again now? I adjusted the modal for Bootstrap 5.
Comment 11 Lucas Gass (lukeg) 2024-09-23 22:58:23 UTC
Oh, something I noticed. I can only change the patron category once. 

-Change to HOMEBOUND, say yes to change the prefs. 
-Change to another patron category, no pop-op and prefs stay the same.
Comment 12 Marcel de Rooy 2024-11-15 08:32:26 UTC
(In reply to Lucas Gass (lukeg) from comment #11)
> Oh, something I noticed. I can only change the patron category once. 
> 
> -Change to HOMEBOUND, say yes to change the prefs. 
> -Change to another patron category, no pop-op and prefs stay the same.

Please clarify or fix. Changing status to reflect need for feedback