X Tutup
The Wayback Machine - https://web.archive.org/web/20200910133231/https://github.com/validatorjs/validator.js/issues/1417
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

Valid phone number from Uzbekistan #1417

Closed
icyice0217 opened this issue Aug 19, 2020 · 5 comments
Closed

Valid phone number from Uzbekistan #1417

icyice0217 opened this issue Aug 19, 2020 · 5 comments

Comments

@icyice0217
Copy link
Contributor

@icyice0217 icyice0217 commented Aug 19, 2020

Hello everyone, we need add 'uz-UZ' locale in isMobilePhone.js lib to do validation for Uzbekistan mobile phones.
Country code: +998
format mobile phones: ( [+998] )? ( ( [6] ( [1] | [2] | [5] | [6] | [7] | [9] ) ) | ( [7] ( [1] | [2] | [3] | [4] | [5] | [6] | [9] ) ) | [88] | [9X] ) [XXX XXXX]
[X] phone number: 7 or 8 digits
I've made an initial expression as:
/^(\+?998)?(6[125-79]|7[1-69]|88|9\d)\d{7}$/
Please improve the expression and add it into lib
for more info about Uzbekistan phone codes see : areaphonecodes
Best Regards

@icyice0217
Copy link
Contributor Author

@icyice0217 icyice0217 commented Aug 20, 2020

@profnandaa can i take this one?

@profnandaa
Copy link
Collaborator

@profnandaa profnandaa commented Aug 20, 2020

@icyice0217 -- sure, please go for it!

@icyice0217
Copy link
Contributor Author

@icyice0217 icyice0217 commented Aug 21, 2020

PR has been created on #1420

@icyice0217
Copy link
Contributor Author

@icyice0217 icyice0217 commented Aug 24, 2020

PR #1420 was merged, thanks guys for all the hard work.

@icyice0217 icyice0217 closed this Aug 24, 2020
@profnandaa
Copy link
Collaborator

@profnandaa profnandaa commented Aug 24, 2020

Thank you too, and welcome to the project!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.
X Tutup