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

Fixes #415 - adds vanity number #683

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anselmbradford
Copy link
Member

Fixes #415 - adds vanity number to phone partial and test spec.

Fixes #415 - adds vanity number to phone partial and test spec.
@anselmbradford
Copy link
Member Author

Looks like:

Location details:
screen shot 2014-11-13 at 7 35 54 pm

Search results view:
screen shot 2014-11-13 at 7 35 48 pm

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling f7a5a80 on 415-vanity-phone-number into 54a1778 on master.

@anselmbradford
Copy link
Member Author

@monfresh Ready for merge. Thx!

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling f7a5a80 on 415-vanity-phone-number into 54a1778 on master.

@monfresh
Copy link
Member

Looks good to me. I'll update the vanity number in the sample data to add a dash after 703 so that it looks like 703-555-ABCD, which is how it's most likely to be entered.

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.

Location details page is not displaying certain phone number attributes
3 participants