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.

Sign up for GitHub

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

Jump to bottom

[18.0][MIG] delivery_carrier_label_batch: Migration to 18.0 #984

Open
wants to merge 61 commits into
base: 18.0
Choose a base branch
from
Open

[18.0][MIG] delivery_carrier_label_batch: Migration to 18.0 #984

wants to merge 61 commits into from

Conversation

yankinmax
Copy link
Contributor

@yankinmax
Copy link
Contributor Author

The tests are failing because of the issue I've mentioned in this PR (delivery_postlogistics)

@imlopes imlopes mentioned this pull request Feb 16, 2025
Closed
@rousseldenis
Copy link
Contributor

/ocabot migration delivery_carrier_label_batch

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Feb 17, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Feb 17, 2025
Open
24 tasks
@StephaneMangin StephaneMangin mentioned this pull request Feb 18, 2025
Open
@yankinmax yankinmax force-pushed the 18.0-mig-delivery_carrier_label_batch branch 4 times, most recently from a17288a to dd119da Compare February 20, 2025 13:52
@yankinmax yankinmax force-pushed the 18.0-mig-delivery_carrier_label_batch branch 3 times, most recently from ca31b7c to e3888d0 Compare February 24, 2025 08:40
@yankinmax yankinmax force-pushed the 18.0-mig-delivery_carrier_label_batch branch from e3888d0 to 823ed3d Compare February 24, 2025 14:29
@StephaneMangin
Copy link

delivery_postlogistics is a final module, why do you need to inherit from it?

@yankinmax
Copy link
Contributor Author

@StephaneMangin if the field parcel_tracking = fields.Char() is defined in postlogistics I need to inherit

yvaucher and others added 9 commits March 27, 2025 11:55
@yvaucher @yankinmax
[9.0] port module delivery_carrier_label_dispatch
6c577d3 
- change name to batch to match with change of `picking_dispatch` name to
`stock_batch_picking`
- mode description in README.rst
- move files in models and views directories
- change headers to small license header
- use api 8.0
- fix and improve tests
@yvaucher @yankinmax
Update translations
aa4732e
@yvaucher @yankinmax
Cannot update options due to limitation of api 8.0
4266d0b
@mdietrichc2c @yankinmax
Use IDs instead of recordset for write
2e1585c
@guewen @yankinmax
rename field tracking_id to package_id
f1b1a5f 
the field has been renamed in the base module
@mdietrichc2c @yankinmax
Remove self.pool + add correct attachment name
fe0c609
@mdietrichc2c @yankinmax
View changed in batch_stock_picking
80546c9
@guewen @yankinmax
Fix condition
bcc7fdf
@guewen @yankinmax
Thread label generation to make it faster
af9b6b4 
Port of OCA#27
OCA-git-bot and others added 29 commits March 27, 2025 11:55
@OCA-git-bot @yankinmax
[UPD] README.rst
45a07bd
@OCA-git-bot @yankinmax
[ADD] icon.png
8075d81
@yvaucher @yankinmax
Remove broken icon
dc3796b
@OCA-git-bot @yankinmax
delivery_carrier_label_batch 12.0.1.0.1
4e518a0
@oca-transbot @yankinmax
Update translation files
d228df6 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: delivery-carrier-12.0/delivery-carrier-12.0-delivery_carrier_label_batch
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-12-0/delivery-carrier-12-0-delivery_carrier_label_batch/
@kv1612 @yankinmax
[IMP] delivery_carrier_label_batch: black, isort, prettier
a654cf2
@kv1612 @yankinmax
[MIG] delivery_carrier_label_batch: Migration to 14.0
83602fd
@oca-travis @yankinmax
[UPD] Update delivery_carrier_label_batch.pot
dc8fbbc
@OCA-git-bot @yankinmax
[UPD] README.rst
56d4b4a
@OCA-git-bot @yankinmax
[IMP] update dotfiles [ci skip]
ea499c0
@StephaneMangin @yankinmax
FIX delivery_carrier_label_batch invalid field
b5f27ce
@grindtildeath @yankinmax
delivery_carrier_label_batch: Allow testing labels generation
2f76ecf 
The threading option doesn't work in test mode but was not tested
because labels were already generated in the test.
Using a condition to use different threads only when test mode
is not activated allows to test label generation.
@grindtildeath @hailangvn @yankinmax
Apply suggestions from code review
953080b 
Co-authored-by: Hai Lang <hailn@trobz.com>
@sebalix @yankinmax
delivery_carrier_label_batch: empty cache once threads finished
9e9ad25
@OCA-git-bot @yankinmax
delivery_carrier_label_batch 14.0.1.0.1
0d09423
@OCA-git-bot @yankinmax
delivery_carrier_label_batch 14.0.1.0.2
174602d
@StephaneMangin @yankinmax
[IMP] delivery_carrier_label_batch: tracking reference purge
c4e4b46 
Sometimes, when regenerating labels from third party API, we could not get the
new tracking references. And then as we do not purge tracking references
correctly, previous ones are kept and lead to incoherences (batch picking
tracking references emptied but labels still existing). For this we need to
manually purge references when asking for full regeneration.
@OCA-git-bot @yankinmax
delivery_carrier_label_batch 14.0.1.1.0
2bc3251
@OCA-git-bot @yankinmax
[UPD] README.rst
0eb79bb
@weblate @yankinmax
Update translation files
6867db8 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: delivery-carrier-14.0/delivery-carrier-14.0-delivery_carrier_label_batch
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-14-0/delivery-carrier-14-0-delivery_carrier_label_batch/
@pedrobaeza @yankinmax
[FIX] *: Linter fixes
2e7505d
@OCA-git-bot @yankinmax
[BOT] post-merge updates
5d10a80
@Ivorra78 @yankinmax
Added translation using Weblate (Spanish)
43a85c3
@Ivorra78 @yankinmax
Translated using Weblate (Spanish)
4b96265 
Currently translated at 2.8% (1 of 35 strings)

Translation: delivery-carrier-14.0/delivery-carrier-14.0-delivery_carrier_label_batch
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-14-0/delivery-carrier-14-0-delivery_carrier_label_batch/es/
@Ivorra78 @yankinmax
Translated using Weblate (Spanish)
f229bbb 
Currently translated at 100.0% (35 of 35 strings)

Translation: delivery-carrier-14.0/delivery-carrier-14.0-delivery_carrier_label_batch
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-14-0/delivery-carrier-14-0-delivery_carrier_label_batch/es/
@mymage @yankinmax
Added translation using Weblate (Italian)
85d8f26
@yankinmax
[IMP] delivery_carrier_label_batch: pre-commit auto fixes
8d506ce
@yankinmax
[MIG] delivery_carrier_label_batch: Migration to 18.0
63f48bb
@yankinmax
DON'T MERGE test requirements
ef34263
@yankinmax yankinmax force-pushed the 18.0-mig-delivery_carrier_label_batch branch from 823ed3d to ef34263 Compare March 27, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@imlopes imlopes imlopes approved these changes

@vvrossem vvrossem vvrossem approved these changes

@Ricardoalso Ricardoalso Ricardoalso approved these changes

Assignees
No one assigned
Labels
approved
Projects
None yet
Milestone
18.0
Development

Successfully merging this pull request may close these issues.