Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Phone format is required to be nnn-nnn-nnnn. #1472

Merged
merged 2 commits into from
Nov 19, 2015

Conversation

biwashingtonial
Copy link
Contributor

Added match/replace options, per #1462 (comment)

@j-ro
Copy link
Contributor

j-ro commented Sep 1, 2015

I don't think match and replace is a supported format here in our schema, unfortunately. If your implementation supports it, great, but I don't think it can be used in this repo. Could use javascript instead if you want.

@biwashingtonial
Copy link
Contributor Author

The match/replace method was per here:
#1462 (comment)

But javascript works as well! Switched methods in the last commit.

@j-ro
Copy link
Contributor

j-ro commented Nov 19, 2015

I know right! Javascript solves all.

j-ro added a commit that referenced this pull request Nov 19, 2015
Phone format is required to be nnn-nnn-nnnn.
@j-ro j-ro merged commit 874fed1 into unitedstates:master Nov 19, 2015
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.

2 participants