Our librarians requested a reminder to unset "gone no address" flag from patron's record once the patron has made a modification request to update their address. I propose adding a message box under patron modification request to notify librarians about patrons that have gone no address flag on, and an option to unset the flag without the need of having to navigate into patron's details.
Created attachment 67524 [details] [review] Bug 19400: Reminder to unset gone no address flag after patron makes a modification request Our librarians requested a reminder to unset "gone no address" flag from patron's record once the patron has made a modification request to update their address. I propose adding a message box under patron modification request to notify librarians about patrons that have gone no address flag on, and an option to unset the flag without the need of having to navigate into patron's details. To test: 1. Apply patch 2. Set "Gone no address" flag for your test patron. You can do this by going to patron modification screen in staff client. 3. Go to OPAC with your test patron 4. Make a modification request for your personal details 5. Go to staff client and see pending modification requests 6. Open the request you just created 7. Observe a message dialog that says this patron has gone no address flag set 8. Check the checkbox to unset the flag and approve the modification request 9. Click Submit 10. Observe your test patron no longer has gone no address flag set 11. Repeat steps 2-7 12. Do not check the checkbox, but approve the modification request 13. Observe your test patron still has gone no address flag set 14. Remove the gone no address flag from your test patron 15. Repeat steps 3-6 16. Observe there is no message dialog for gone no address
Created attachment 67575 [details] [review] Bug 19400: Reminder to unset gone no address flag after patron makes a modification request Our librarians requested a reminder to unset "gone no address" flag from patron's record once the patron has made a modification request to update their address. I propose adding a message box under patron modification request to notify librarians about patrons that have gone no address flag on, and an option to unset the flag without the need of having to navigate into patron's details. To test: 1. Apply patch 2. Set "Gone no address" flag for your test patron. You can do this by going to patron modification screen in staff client. 3. Go to OPAC with your test patron 4. Make a modification request for your personal details 5. Go to staff client and see pending modification requests 6. Open the request you just created 7. Observe a message dialog that says this patron has gone no address flag set 8. Check the checkbox to unset the flag and approve the modification request 9. Click Submit 10. Observe your test patron no longer has gone no address flag set 11. Repeat steps 2-7 12. Do not check the checkbox, but approve the modification request 13. Observe your test patron still has gone no address flag set 14. Remove the gone no address flag from your test patron 15. Repeat steps 3-6 16. Observe there is no message dialog for gone no address Followed test plan, patch worked as described. Also ran QA test tools and all modified files passed Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>
Created attachment 68753 [details] [review] Bug 19400: Reminder to unset gone no address flag after patron makes a modification request Our librarians requested a reminder to unset "gone no address" flag from patron's record once the patron has made a modification request to update their address. I propose adding a message box under patron modification request to notify librarians about patrons that have gone no address flag on, and an option to unset the flag without the need of having to navigate into patron's details. To test: 1. Apply patch 2. Set "Gone no address" flag for your test patron. You can do this by going to patron modification screen in staff client. 3. Go to OPAC with your test patron 4. Make a modification request for your personal details 5. Go to staff client and see pending modification requests 6. Open the request you just created 7. Observe a message dialog that says this patron has gone no address flag set 8. Check the checkbox to unset the flag and approve the modification request 9. Click Submit 10. Observe your test patron no longer has gone no address flag set 11. Repeat steps 2-7 12. Do not check the checkbox, but approve the modification request 13. Observe your test patron still has gone no address flag set 14. Remove the gone no address flag from your test patron 15. Repeat steps 3-6 16. Observe there is no message dialog for gone no address Followed test plan, patch worked as described. Also ran QA test tools and all modified files passed Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Pushed to master for 17.11, thanks to everybody involved!