Bug 39642

Summary: Add SMS number to hold found modals on return.tt
Product: Koha Reporter: Lucas Gass (lukeg) <lucas>
Component: CirculationAssignee: Lucas Gass (lukeg) <lucas>
Status: Signed Off --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: baptiste.wojtkowski, gmcharlt, kyle.m.hall
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 39642: Add SMS number to holds modals on return.tt
Bug 39642: Add SMS number to holds modals on return.tt
Bug 39642: Add SMS number to holds modals on return.tt

Description Lucas Gass (lukeg) 2025-04-15 16:55:05 UTC
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.
Comment 1 Lucas Gass (lukeg) 2025-04-15 16:58:20 UTC
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.
Comment 2 Owen Leonard 2025-04-15 19:41:11 UTC
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>
Comment 3 Lucas Gass (lukeg) 2025-06-27 13:59:38 UTC
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>
Comment 4 Lucas Gass (lukeg) 2025-06-27 14:00:02 UTC
rebased for current main