Skip to content

feat(notifications): require mobile number when SMS method selected#8168

Open
saewitz wants to merge 2 commits into
developfrom
jsaewitz/otr-1875-if-user-selects-sms-notifications-force-phone-number-entry
Open

feat(notifications): require mobile number when SMS method selected#8168
saewitz wants to merge 2 commits into
developfrom
jsaewitz/otr-1875-if-user-selects-sms-notifications-force-phone-number-entry

Conversation

@saewitz

@saewitz saewitz commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • When staff selects "Phone" or "Phone and Computer" as their notification method, the Mobile field is now marked as required
  • Clicking "Save changes" without a phone number turns the field red with "Required" helper text (previously showed a generic snackbar that was easy to miss)
  • Renames the field label from "Phone" to "Mobile"
  • Clears the field error as soon as the user starts typing or switches to a non-phone method

Closes OTR-1875

Test plan

  • Go to My Profile → Notification Settings
  • Enable task or telemed notifications and set "Notify me by" to "Phone" or "Phone and Computer"
  • Confirm "Mobile" field shows a required asterisk (*)
  • Click "Save changes" with the Mobile field empty — field should turn red with "Required" helper text, save should not proceed
  • Enter a valid phone number, click "Save changes" — should save successfully
  • Set method to "Computer" — Mobile field should disable and the red error (if present) should clear
  • Start typing a phone number while the field is red — error should clear immediately

🤖 Generated with Claude Code

saewitz and others added 2 commits June 12, 2026 17:28
If staff selects 'Phone' or 'Phone and Computer' as their notification
method, the Mobile field is now required. Clicking 'Save changes' without
a phone number turns the field red with a 'Required' helper text instead
of showing a generic snackbar. Renames the field label from 'Phone' to
'Mobile' per OTR-1875.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant