Bug 39642 - Add SMS number to hold found modals on return.tt
Summary: Add SMS number to hold found modals on return.tt
Status: Pushed to main
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Lucas Gass (lukeg)
QA Contact: Laura Escamilla
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-04-15 16:55 UTC by Lucas Gass (lukeg)
Modified: 2025-10-22 13:44 UTC (History)
4 users (show)

See Also:
GIT URL:
Initiative type: ---
Sponsorship status: ---
Comma delimited list of Sponsors:
Crowdfunding goal: 0
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Enhancement: The “Hold for:” section in the waiting-hold and transfer-hold modals now displays the patron’s SMS alert number (if present). This makes it easier for staff to quickly see contact details when processing holds.
Version(s) released in:
25.11.00
Circulation function:


Attachments
Bug 39642: Add SMS number to holds modals on return.tt (2.30 KB, patch)
2025-04-15 16:58 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review
Bug 39642: Add SMS number to holds modals on return.tt (2.35 KB, patch)
2025-04-15 19:41 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 39642: Add SMS number to holds modals on return.tt (2.49 KB, patch)
2025-06-27 13:59 UTC, Lucas Gass (lukeg)
Details | Diff | Splinter Review
Bug 39642: Add SMS number to holds modals on return.tt (2.56 KB, patch)
2025-09-30 12:39 UTC, Laura Escamilla
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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
Comment 5 Laura Escamilla 2025-09-30 12:39:33 UTC
Created attachment 187097 [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>
Signed-off-by: Laura_Escamilla <laura.escamilla@bywatersolutions.com>
Comment 6 Lucas Gass (lukeg) 2025-10-22 13:44:56 UTC
Nice work everyone!

Pushed to main for 25.11