entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 39 Releases Activity

misc: add validation for local mobile phone number (#73498) #325

Merged
fpeters merged 1 commits from wip/73498-mobile-validation into main 2023-05-26 07:43:10 +02:00
Conversation 1 Commits 1 Files Changed 3 +76
fpeters commented 2023-05-18 17:53:44 +02:00
Owner
Copy Link
No description provided.
fpeters commented 2023-05-18 19:03:01 +02:00
Author
Owner
Copy Link

Je parlais dans le ticket d'intégrer ça via une nouvelle configuration de pays par défaut, via hobo, mais on a déjà introduit un paramètre default-country-code pour les bouts de normalisation des numéros de téléphone, donc on peut juste réutiliser ça ici.

Je parlais dans le ticket d'intégrer ça via une nouvelle configuration de pays par défaut, via hobo, mais on a déjà introduit un paramètre default-country-code pour les bouts de normalisation des numéros de téléphone, donc on peut juste réutiliser ça ici.
fpeters force-pushed wip/73498-mobile-validation from 8a5704c8e4 to af3b1ec1aa 2023-05-18 19:06:35 +02:00 Compare
fpeters reviewed 2023-05-18 19:08:27 +02:00
wcs/qommon/misc.py
@ -857,0 +858,4 @@
region_code = get_publisher().get_site_option('default-country-code') or 'FR'
country_codes = []
if region_code == 'FR':
country_codes = [33, 262, 508, 590, 594, 596]
fpeters commented 2023-05-18 19:08:22 +02:00
Author
Owner
Copy Link

Cas particulier ici pour être similaire à ce qu'on fait pour la validation "téléphone (français)".

Cas particulier ici pour être similaire à ce qu'on fait pour la validation "téléphone (français)".
wcs/qommon/misc.py
@ -857,0 +872,4 @@
return bool(
phonenumbers.is_valid_number(pn)
and phonenumbers.number_type(pn)
in (phonenumbers.PhoneNumberType.MOBILE, phonenumbers.PhoneNumberType.FIXED_LINE_OR_MOBILE)
fpeters commented 2023-05-18 19:07:37 +02:00
Author
Owner
Copy Link

Il y a des pays où ça n'est pas possible de distinguer le côté mobile, c'est là que vient FIXED_LINE_OR_MOBILE :

    # In some regions (e.g. the USA), it is impossible to distinguish between
    # fixed-line and mobile numbers by looking at the phone number itself.
    FIXED_LINE_OR_MOBILE = 2
Il y a des pays où ça n'est pas possible de distinguer le côté mobile, c'est là que vient FIXED_LINE_OR_MOBILE : ``` # In some regions (e.g. the USA), it is impossible to distinguish between # fixed-line and mobile numbers by looking at the phone number itself. FIXED_LINE_OR_MOBILE = 2 ```
fpeters changed title from WIP: misc: add validation for local mobile phone number (#73498) to misc: add validation for local mobile phone number (#73498) 2023-05-18 19:08:43 +02:00
lguerin approved these changes 2023-05-25 14:30:01 +02:00
fpeters merged commit 1147da7e3a into main 2023-05-26 07:43:10 +02:00
fpeters deleted branch wip/73498-mobile-validation 2023-05-26 07:43:10 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#325
No description provided.
Delete Branch "wip/73498-mobile-validation"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?