delivery-carrier icon indicating copy to clipboard operation
delivery-carrier copied to clipboard

[16.0][FIX] delivery_postlogistics: splitting package label by package

Open imlopes opened this issue 10 months ago • 2 comments

Problem :

In a picking, if we create 2 or more packages, after generating the postlogistic labels, all tracking numbers are concatenated on the first package.

image

image

It seems that this behavior was added here : https://github.com/OCA/delivery-carrier/commit/5504c9f26dae2739abb68314bab7ffb2756f332d

More precisely here : https://github.com/OCA/delivery-carrier/blob/5504c9f26dae2739abb68314bab7ffb2756f332d/delivery_postlogistics/postlogistics/web_service.py#L511C1-L518C14

and here : https://github.com/OCA/delivery-carrier/pull/323/files#diff-d3c06dad2d58ebdeb558386909ed5b2ad9ed22d84b8476e5fb2658ae4308d61f

I didn't identify why the code was changed before versions, but now we are always taking the first package when preparing the label values that will be used on write_tracking_number_label method.

imlopes avatar Apr 16 '24 18:04 imlopes

@imlopes Thanks for this. Could you maybe add a test to avoid regression ?

rousseldenis avatar Apr 17 '24 06:04 rousseldenis

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot avatar Apr 25 '24 08:04 OCA-git-bot

/ocabot merge patch

dreispt avatar May 18 '24 16:05 dreispt

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-798-by-dreispt-bump-patch, awaiting test results.

OCA-git-bot avatar May 18 '24 16:05 OCA-git-bot

Congratulations, your PR was merged at 68fcaf0b00bb6b0ca3d90729eb9301056c488f5d. Thanks a lot for contributing to OCA. ❤️

OCA-git-bot avatar May 18 '24 16:05 OCA-git-bot