Skip to content

fix(isMobilePhone): add dutch phone number format - #923

Closed
ghost wants to merge 1 commit into
masterfrom
unknown repository
Closed

fix(isMobilePhone): add dutch phone number format#923
ghost wants to merge 1 commit into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Nov 3, 2018

Copy link
Copy Markdown

Hi, love your library! Here's dutch phone number. Let me know do you need anything else to be changed in this pr to merge :)

@chriso

chriso commented Nov 4, 2018

Copy link
Copy Markdown
Collaborator

Thank you! 😄

Do you mind updating the README and adding some unit tests?

@chriso chriso added the 🧹 needs-update For PRs that need to be updated before landing label Nov 4, 2018
@profnandaa

Copy link
Copy Markdown
Member

@odrzutowiec - ping!

@mayurssoni2456

Copy link
Copy Markdown

Need support for Dutch locale. Not sure why this PR is open. Please help.

@profnandaa

Copy link
Copy Markdown
Member

@mayurssoni2456 -- cleaning this up, should be in the next release, in a few weeks.

profnandaa added a commit to profnandaa/validator.js that referenced this pull request Jul 2, 2019
@profnandaa profnandaa mentioned this pull request Jul 2, 2019
@profnandaa profnandaa closed this in 1a2a868 Jul 2, 2019
@ghost

ghost commented Jul 2, 2019

Copy link
Copy Markdown
Author

Yes, sorry guys I didn't follow up on this. 🙈 Good luck!

@profnandaa

Copy link
Copy Markdown
Member

@odrzutowiec -- no worries :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧹 needs-update For PRs that need to be updated before landing stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants