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

Add Egypt locale for phone numbers #1119

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ahmeda335
Copy link

Adding Regex for Phone Numbers in EGYPT

Adding Regex for Phone Numbers in EGYPT
@cpfiffer
Copy link
Contributor

This might actually be a good time to look at using something more general. I was able to get phonenumbers to spit out the regex(s):

import phonenumbers
from phonenumbers.phonemetadata import PhoneMetadata

metadata = PhoneMetadata.metadata_for_region("EG")  # Replace "US" with the country code you need

metadata.number_format

# [NumberFormat(pattern='(\\d)(\\d{7,8})', format='\\1 \\2', leading_digits_pattern=['[23]'], national_prefix_formatting_rule='0\\1'), NumberFormat(pattern='(\\d{2})(\\d{6,7})', format='\\1 \\2', leading_digits_pattern=['1[35]|[4-6]|8[2468]|9[235-7]'], national_prefix_formatting_rule='0\\1'), NumberFormat(pattern='(\\d{3})(\\d{3})(\\d{4})', format='\\1 \\2 \\3', leading_digits_pattern=['[89]'], national_prefix_formatting_rule='0\\1'), NumberFormat(pattern='(\\d{2})(\\d{8})', format='\\1 \\2', leading_digits_pattern=['1'], national_prefix_formatting_rule='0\\1')]

Not quite sure how to combine these though.

@rlouf
Copy link
Member

rlouf commented Sep 17, 2024

Thank you for your contribution! You need to add the Egypt locale in this file:

so users can call:

types.locale("eg").PhoneNumber

@rlouf rlouf changed the title Update phone_numbers.py Add Egypt locale for phone numbers Sep 17, 2024
@rlouf rlouf added this to the 0.1.3 milestone Nov 8, 2024
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.

3 participants