If a waiting hold is triggered a popup appears that includes "Patron notification:". This is based on the patron messaging preferences for 'Hold filled'. We should allow libraries to choose between having this based on messaging preferences or primary_contact_method.
Hi Lucas, I am not sure if I understand this one. I think the idea of the display is to show if an email notification was sent out or not. The primary contact method from the patron record currently does have no effect and is only for display. Maybe it could be shown additionally instead of instead?
(In reply to Katrin Fischer from comment #1) > Hi Lucas, > > I am not sure if I understand this one. > > I think the idea of the display is to show if an email notification was sent > out or not. The primary contact method from the patron record currently does > have no effect and is only for display. > > Maybe it could be shown additionally instead of instead? Showing it additionally would be fine with me. Changing the title of this bug to reflect that.
Created attachment 149136 [details] [review] Bug 33398: Add primary_contact_method to display when triggering a waiting hold To test: 1. Apply patch, yarn build 2. Trigger a waiting hold for a patron that has a value for primary_contact_method 3. Notice the modal now includes a line indicating the primary_contact_method. 4. Trigger a hold for a patron without a primary_contact_method. Nothing should display.
Created attachment 149202 [details] [review] Bug 33398: Add primary_contact_method to display when triggering a waiting hold To test: 1. Apply patch, yarn build 2. Trigger a waiting hold for a patron that has a value for primary_contact_method 3. Notice the modal now includes a line indicating the primary_contact_method. 4. Trigger a hold for a patron without a primary_contact_method. Nothing should display. Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com>
Created attachment 150399 [details] [review] Bug 33398: Add primary_contact_method to display when triggering a waiting hold To test: 1. Apply patch, yarn build 2. Trigger a waiting hold for a patron that has a value for primary_contact_method 3. Notice the modal now includes a line indicating the primary_contact_method. 4. Trigger a hold for a patron without a primary_contact_method. Nothing should display. Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Pushed to master for 23.11. Nice work everyone, thanks!
Enhancement, no backport for 23.05.x