misc: rely on phonenumbers library for is_french_mobile_phone_number (#72793) #1037

Open
fpeters wants to merge 1 commits from wip/72793-french-mobile-number-templatetag into main
Owner
No description provided.
fpeters added 1 commit 2024-01-16 19:21:22 +01:00
fpeters changed title from WIP: misc: rely on phonenumbers library for is_french_mobile_phone_number (#72793) to misc: rely on phonenumbers library for is_french_mobile_phone_number (#72793) 2024-01-18 18:11:46 +01:00
tnoel requested changes 2024-01-19 12:02:58 +01:00
tnoel left a comment
Owner

A priori on pourrait donc faire évoluer test_is_french_mobile_phone_number pour tester l'usage de numéros mobiles avec numérotation internationale, genre +33 6 39 98 42 42 et +262 6 92 42 42 42 qui doivent être ok, alors qu'un +61 442 424242 (mobile australie) ne devrait toujours pas passer.

A priori on pourrait donc faire évoluer test_is_french_mobile_phone_number pour tester l'usage de numéros mobiles avec numérotation internationale, genre +33 6 39 98 42 42 et +262 6 92 42 42 42 qui doivent être ok, alors qu'un +61 442 424242 (mobile australie) ne devrait toujours pas passer.
Author
Owner

A priori on pourrait donc faire évoluer test_is_french_mobile_phone_number

Peut-être mais je serais très frileux, en imaginant trop facilement que soudainement des numéros internationaux vont être envoyés à une application métier qui ne s'y attend pas; et je préfère donc rester prudent ici.

> A priori on pourrait donc faire évoluer test_is_french_mobile_phone_number Peut-être mais je serais très frileux, en imaginant trop facilement que soudainement des numéros internationaux vont être envoyés à une application métier qui ne s'y attend pas; et je préfère donc rester prudent ici.
fpeters requested review from tnoel 2024-01-19 13:14:26 +01:00
All checks were successful
gitea/wcs/pipeline/head This commit looks good
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#1037
No description provided.