Skip to content

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Mar 18, 2025

Current behaviour:

Mexican phone numbers are not managed correctly following the 2019 changes in Mexico.
(Removing a 1 in phone numbers)

Cause of the issue:

The phonenumbers library removing support of mexican phone numbers starting with 1 in 8.13.38
The Whatsapp API still using the mexican phone numbers starting with 1

Fixes:

Patched the phonenumbers library, removing 1 at the right place for mexican phone numbers.
Similar to: 53885e4

Changed the forcing to international format when parsing, to make sure the patches are applied before detecting an issue.
Similar to: 29a4de8

opw-4473528


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #197838

@robodoo
Copy link
Contributor

robodoo commented Mar 18, 2025

Pull request status dashboard

@robodoo robodoo added forwardport This PR was created by @fw-bot conflict There was an error while creating this forward-port PR labels Mar 18, 2025
@fw-bot
Copy link
Contributor Author

fw-bot commented Mar 18, 2025

@ande-odoo @tde-banana-odoo cherrypicking of pull request #197838 failed.

stdout:

Auto-merging addons/phone_validation/tools/phone_validation.py
CONFLICT (content): Merge conflict in addons/phone_validation/tools/phone_validation.py

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

In the former case, you may want to edit this PR message as well.

⚠️ after resolving this conflict, you will need to merge it via @robodoo.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Mar 18, 2025
Current behaviour:
Mexican phone numbers are not managed correctly
following the 2019 changes in Mexico.
(Removing a 1 in phone numbers)

Cause of the issue:
The phonenumbers library removing support of mexican phone numbers
starting with 1 in 8.13.38
The Whatsapp API still using the mexican phone numbers starting with 1

Fixes:
Patched the phonenumbers library, removing 1
at the right place for mexican phone numbers.
Similar to: odoo@53885e4

Changed the forcing to international format when parsing, to make
sure the patches are applied before detecting an issue.
Similar to: odoo@29a4de8

opw-4473528

X-original-commit: 8452b3a
@ande-odoo ande-odoo force-pushed the 18.0-17.0-OPW_4473528-phone_validation-mexican_phone_numbers-ande-423732-fw branch from 253e478 to 5a68f39 Compare March 18, 2025 13:31
@C3POdoo C3POdoo requested a review from a team March 18, 2025 13:43
@ande-odoo
Copy link
Contributor

@robodoo r+

@robodoo robodoo closed this in 9ccc52b Mar 19, 2025
omar-sherif9992 pushed a commit to odoo-dev/odoo that referenced this pull request Mar 26, 2025
Current behaviour:
Mexican phone numbers are not managed correctly
following the 2019 changes in Mexico.
(Removing a 1 in phone numbers)

Cause of the issue:
The phonenumbers library removing support of mexican phone numbers
starting with 1 in 8.13.38
The Whatsapp API still using the mexican phone numbers starting with 1

Fixes:
Patched the phonenumbers library, removing 1
at the right place for mexican phone numbers.
Similar to: odoo@53885e4

Changed the forcing to international format when parsing, to make
sure the patches are applied before detecting an issue.
Similar to: odoo@29a4de8

opw-4473528

closes odoo#202295

X-original-commit: 8452b3a
Related: odoo/enterprise#81677
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
Signed-off-by: Antoine Demany (ande) <ande@odoo.com>
@fw-bot fw-bot deleted the 18.0-17.0-OPW_4473528-phone_validation-mexican_phone_numbers-ande-423732-fw branch March 26, 2025 15:05
gamarino pushed a commit to numaes/numa-public-odoo that referenced this pull request Mar 27, 2025
Current behaviour:
Mexican phone numbers are not managed correctly
following the 2019 changes in Mexico.
(Removing a 1 in phone numbers)

Cause of the issue:
The phonenumbers library removing support of mexican phone numbers
starting with 1 in 8.13.38
The Whatsapp API still using the mexican phone numbers starting with 1

Fixes:
Patched the phonenumbers library, removing 1
at the right place for mexican phone numbers.
Similar to: odoo/odoo@53885e4

Changed the forcing to international format when parsing, to make
sure the patches are applied before detecting an issue.
Similar to: odoo/odoo@29a4de8

opw-4473528

closes odoo/odoo#202295

X-original-commit: 8452b3ae74450010cbcd6b84df4b4a4dea6d8380
Related: odoo/enterprise#81677
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
Signed-off-by: Antoine Demany (ande) <ande@odoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflict There was an error while creating this forward-port PR forwardport This PR was created by @fw-bot OE the report is linked to a support ticket (opw-...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants