Skip to content

MPDX-9387 and MPDX-9400 - Allow international phone numbers (7–15 digits)#1703

Merged
canac merged 1 commit intomainfrom
MPDX-9400-invalid-phone
Apr 10, 2026
Merged

MPDX-9387 and MPDX-9400 - Allow international phone numbers (7–15 digits)#1703
canac merged 1 commit intomainfrom
MPDX-9400-invalid-phone

Conversation

@zweatshirt
Copy link
Copy Markdown
Contributor

@zweatshirt zweatshirt commented Apr 10, 2026

Description

Allow phone numbers between lengths 7 and 15 to support international numbers, E.164 standard and trunk prefixes. Remove requirement that the area code is US only.

MPDX-9400
MPDX-9387

Testing

  • Go to the different reports
  • Enter a phone number wherever applicable
  • Check that the validation on the phone works for unique phones (e.g. numbers of length 11 starting with a 0).

Checklist:

  • I have given my PR a title with the format "MPDX-(JIRA#) (summary sentence max 80 chars)"
  • I have applied the appropriate labels (Add the label "Preview" to automatically create a preview environment)
  • I have run the Claude Code /pr-review command locally and fixed any relevant suggestions
  • I have requested a review from another person on the project
  • I have tested my changes in preview or in staging
  • I have cleaned up my commit history

@zweatshirt zweatshirt self-assigned this Apr 10, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

Bundle sizes [mpdx-react]

Compared against 3abca3f

No significant changes found

@zweatshirt zweatshirt added the Preview Environment Add this label to create an Amplify Preview label Apr 10, 2026
@zweatshirt zweatshirt changed the title Allow international phone numbers (7–15 digits) MPDX-9387 and MPDX-9400 - Allow international phone numbers (7–15 digits) Apr 10, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Preview branch generated at https://MPDX-9400-invalid-phone.d3dytjb8adxkk5.amplifyapp.com

@zweatshirt zweatshirt marked this pull request as ready for review April 10, 2026 18:44
@zweatshirt zweatshirt requested a review from canac April 10, 2026 18:52
Copy link
Copy Markdown
Contributor

@canac canac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this! Should we add comments to the two regex tests to explain what they are checking for?

@zweatshirt zweatshirt force-pushed the MPDX-9400-invalid-phone branch from b014f1a to 9447de5 Compare April 10, 2026 19:12
@zweatshirt zweatshirt enabled auto-merge April 10, 2026 19:12
@canac canac disabled auto-merge April 10, 2026 19:13
@canac canac enabled auto-merge April 10, 2026 19:13
@canac canac merged commit 3c5bc76 into main Apr 10, 2026
24 checks passed
@canac canac deleted the MPDX-9400-invalid-phone branch April 10, 2026 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Preview Environment Add this label to create an Amplify Preview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants