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 supports china mobile numbers beginning with 162. #1655

Closed
guoquanwei opened this issue Apr 25, 2021 · 6 comments
Closed

isMobilePhone supports china mobile numbers beginning with 162. #1655

guoquanwei opened this issue Apr 25, 2021 · 6 comments

Comments

@guoquanwei
Copy link

Describe the bug
In Jilin, China, there are mobile phone numbers beginning with 162.
【isMobilePhone】'zh-CN': /^((+|00)86)?1([358][0-9]|4[579]|6[67]|7[0135678]|9[189])[0-9]{8}$/;
It should be 【 'zh-CN': /^((+|00)86)?1([358][0-9]|4[579]|6[267]|7[0135678]|9[189])[0-9]{8}$/ 】.

@tux-tn
Copy link
Member

tux-tn commented Apr 25, 2021

Thank you for reporting this! PR welcome

@erana111
Copy link

erana111 commented May 5, 2021

Hi @tux-tn, I can create a PR for it :)

@erana111
Copy link

erana111 commented May 5, 2021

@guoquanwei I can't recreate the bug, please give a valid and non-valid Chinese phone numbers for tests example

@laulujan
Copy link
Contributor

laulujan commented Jul 9, 2021

Hello, I would like to give it a try, I'll submit a PR for this issue!!

@parasg1999
Copy link
Contributor

Looks like this has been resolved, it would be great if someone could close this :)

@tux-tn
Copy link
Member

tux-tn commented Oct 1, 2021

Closed in #1695
Thank you @parasg1999 for the reminder

@tux-tn tux-tn closed this as completed Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants