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

Fix some invalid French phone numbers starting with "+33 8x" #933

Merged
merged 1 commit into from
Mar 26, 2019
Merged

Fix some invalid French phone numbers starting with "+33 8x" #933

merged 1 commit into from
Mar 26, 2019

Conversation

stephane
Copy link
Contributor

The full range of phone number starting with 08 isn't valid (eg. 083xx is not valid) so this patch limits the range to valid numbers only.

French validator used by the phonenumbers project:
https://github.com/daviddrysdale/python-phonenumbers/blob/dev/python/phonenumbers/data/region_FR.py#L9

What does this changes

Limits the possible French phone numbers.

What was wrong

+3383xxx isn't valid.

How this fixes it

Patch provided.

The full range of phone number starting with 08 isn't valid (eg. 083xx is not valid) so this patch limits the range to valid numbers only.

French validator used by the phonenumbers project:
https://github.com/daviddrysdale/python-phonenumbers/blob/dev/python/phonenumbers/data/region_FR.py#L9
@fcurella
Copy link
Collaborator

Thank you! ✨

@fcurella fcurella merged commit 4cc1185 into joke2k:master Mar 26, 2019
@stephane stephane deleted the patch-1 branch March 26, 2019 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants