Skip to content

Commit e4e925c

Browse files
fix: Hide checkbox for text messages (#182)
DMS does not want to deal with the effort needed for sending SMS at this time, and this checkbox confuses people/gives them a false sense. It is being hidden, because we might want it back at some point, and I didn't want to make a DB change to set this field non required.
1 parent 7e13e5f commit e4e925c

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

src/Template/Registrations/event.ctp

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,12 @@
4141
'default' => ($authUser ? $authUser['mail'] : '')
4242
]) ?>
4343
<?= $this->Form->input('phone', [
44-
'default' => ($authUser ? $authUser['telephonenumber'] : '')
44+
'default' => ($authUser ? $authUser['telephonenumber'] : ''),
45+
'help' => 'DMS does not send messages to this number, but the teacher will see it and may reach out in the case of a class delay/etc.'
4546
]) ?>
46-
<?= $this->Form->input('send_text', ['label' => 'Receive text message alerts and updates regarding this event.']) ?>
47-
47+
<div style="display: none">
48+
<?= $this->Form->input('send_text', ['label' => 'Receive text message alerts and updates regarding this event.']) ?>
49+
</div>
4850
<?php if ($event->advisories): ?>
4951
<div class="alert alert-warning">
5052
<h4 style="margin-top: 10px"><strong>Special Considerations and Warnings</strong></h4>

0 commit comments

Comments
 (0)