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

isMobilePhone not valid number phone in VietNam #813

Closed
tanmynguyen opened this issue Apr 12, 2018 · 6 comments · Fixed by #1689
Closed

isMobilePhone not valid number phone in VietNam #813

tanmynguyen opened this issue Apr 12, 2018 · 6 comments · Fixed by #1689

Comments

@tanmynguyen
Copy link

Hi @chriso .
I tried validator with number phone on VietNam with locate code vi-VN. But Number Mobile Phone changed on VietNam. With start number phone 08 so isMobilePhone not working with number phone "0868891902". You can help to check Regex of isMobilePhone.
Example:
validator.isMobilePhone("0868891902", "vi-VN") --> false
validator.isMobilePhone("01646545327", "vi-VN") --> true
Thanks You 👍

@tanmynguyen
Copy link
Author

tanmynguyen commented Apr 12, 2018

I test with regx /^(\+?84|0)?((1(2([0-9])|6([2-9])|88|86|99))|((8|9)((?!5)[0-9])))([0-9]{7})$/ everything ok!
// viettel

0961111111
0971111111
0981111111
01621111111
01631111111
01641111111
01651111111
01661111111
01671111111
01681111111
01691111111
0868891905

// Mobile

0903803765
0933803765
01203803765
01213803765
01223803765
01263803765
01283803765

// Vina

0913803765
0943803765
01233803765
01243803765
01253803765
01273803765
01293803765
0888891905

// GMobile

0993803765
0994803765
0995803765
0996803765
0997803765
01999803765

// Vietnam Mobile

0923803765
01861111111
01881111111

https://regexr.com/3nor2

@navata
Copy link

navata commented Oct 4, 2018

VietNam is change 11 number to 10 number phone. I think validator should update for new phone.

@profnandaa
Copy link
Member

@navata - PR welcome :)

@profnandaa
Copy link
Member

profnandaa commented Dec 3, 2018 via email

@ezkemboi
Copy link
Member

I was following up on this issue, and I have realized that the issue being raised does not occur.
Cc. @tanmynguyen, please do confirm so that the issue can be closed.

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

Successfully merging a pull request may close this issue.

6 participants
@profnandaa @chriso @tanmynguyen @navata @ezkemboi and others