Opac. SMS and Carrier Fields display regardless of OU Setting when Editing a Hold

Bug #1902302 reported by Garry Collum
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.6
Fix Released
Medium
Unassigned

Bug Description

When editing an individual hold in the Opac, the SMS Notification field and Default Mobile Carrier Drop-down list display regardless of the 'Enable features that send SMS text messages' setting in the Library Settings Editor.

I believe by default this setting is set to False, but to verify, set the setting to False.

Current master branch.

Garry Collum (gcollum)
description: updated
Revision history for this message
Garry Collum (gcollum) wrote :
tags: added: pullrequest
Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
Changed in evergreen:
assignee: nobody → Terran McCanna (tmccanna)
Changed in evergreen:
assignee: Terran McCanna (tmccanna) → nobody
Revision history for this message
Kyle Huckins (khuckins) wrote :

Thanks Garry! Does what it says on the tin. I have tested this code and consent to signing off on it with my name, Kyle Huckins, and my email address, <email address hidden>

tags: added: signedoff
Michele Morgan (mmorgan)
Changed in evergreen:
milestone: none → 3.6.2
Changed in evergreen:
milestone: 3.6.2 → 3.6.3
Changed in evergreen:
milestone: 3.6.3 → 3.6.4
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed down to rel_3_6. Thanks, Garry and Kyle!

Noting that this issue was not present in the Bootstrap templates, just TPAC.

Changed in evergreen:
milestone: 3.6.4 → 3.7.1
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.