Skip to content

Strip non-numeric characters from phone number - #23931

Merged
tuchfarber merged 1 commit into
masterfrom
tuchfarber/strip_phone_number
May 7, 2020
Merged

Strip non-numeric characters from phone number#23931
tuchfarber merged 1 commit into
masterfrom
tuchfarber/strip_phone_number

Conversation

@tuchfarber

Copy link
Copy Markdown
Contributor

Allows user to type in phone number in whichever format they prefer.

@tuchfarber
tuchfarber requested a review from a team May 6, 2020 23:10

@justinhynes justinhynes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm! 👍

Allows user to type in phone number in whichever
format they prefer.
@tuchfarber
tuchfarber force-pushed the tuchfarber/strip_phone_number branch from f652a9f to 4910cfa Compare May 7, 2020 14:10
@tuchfarber

Copy link
Copy Markdown
Contributor Author

jenkins run a11y

@edx-status-bot

Copy link
Copy Markdown

Your PR has finished running tests. There were no failures.

@tuchfarber
tuchfarber merged commit 8193763 into master May 7, 2020
@tuchfarber
tuchfarber deleted the tuchfarber/strip_phone_number branch May 7, 2020 15:40
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

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.

4 participants