There are 2 holds modal ( waiting hold, hold that needs transfer ) that include the 'Hold for:' text. Some would find it hand if the SMSalertnumber was included here.
Created attachment 180977 [details] [review] Bug 39642: Add SMS number to holds modals on return.tt To test: 0. APPLY PACTH 1. Set SMSSendDriver to test. 2. Find a patron record and add the smsalertnumber 3. Place a hold for that patron. 4. Trigger the hold when it is waiting. 5. Notice the SMS number is included in the Hold for: section of the modal. 6. Change branches so the hold will need a transfer. 7. Check it in, and notice the SMS number is again included in the Hold for: section of the modal.
Created attachment 180994 [details] [review] Bug 39642: Add SMS number to holds modals on return.tt To test: 0. APPLY PACTH 1. Set SMSSendDriver to test. 2. Find a patron record and add the smsalertnumber 3. Place a hold for that patron. 4. Trigger the hold when it is waiting. 5. Notice the SMS number is included in the Hold for: section of the modal. 6. Change branches so the hold will need a transfer. 7. Check it in, and notice the SMS number is again included in the Hold for: section of the modal. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 183605 [details] [review] Bug 39642: Add SMS number to holds modals on return.tt To test: 0. APPLY PACTH 1. Set SMSSendDriver to test. 2. Find a patron record and add the smsalertnumber 3. Place a hold for that patron. 4. Trigger the hold when it is waiting. 5. Notice the SMS number is included in the Hold for: section of the modal. 6. Change branches so the hold will need a transfer. 7. Check it in, and notice the SMS number is again included in the Hold for: section of the modal. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
rebased for current main