-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
feat(isMobilePhone): add support for Philippine mobile no #1388
Conversation
Add support for Philippine mobile no.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check Contributing guidelines and add the missing parts:
- Adding your code to
src/lib
and notlib
- Writing test cases for your new validation
- Adding an entry in README file for your new locale
Hi @tux-tn done with the changes but it won't allow me to re-request review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@stinkymonkeyph thank you for making the necessary changes, you don't need to create a new pull request. I see that your regex is validating numbers starting with +63 followed by 10 decimals, isn't that the international format of philippine mobile numbers? Can you add the format for domestic callers as well?
Yeah sure, I'll add them as well. |
Hi sorry for the late update, been busy with several things. I'm using this package in one of our projects and it was missing ph support. Thank guys for all the good work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎉
Great, looking forward to seeing it on next build :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM too, thanks for your contrib! 🎉
Add support for Philippine mobile no.