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
+2,859 −0

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
23 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 2 times, most recently from 39b2255 to ca31b7c Compare February 21, 2025 07:47
yvaucher and others added 15 commits February 24, 2025 10:39
@yvaucher @yankinmax
[9.0] port module delivery_carrier_label_dispatch
a10f762 
- 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
170f098
@yvaucher @yankinmax
Cannot update options due to limitation of api 8.0
7b74638
@mdietrichc2c @yankinmax
Use IDs instead of recordset for write
5d95958
@guewen @yankinmax
rename field tracking_id to package_id
4715354 
the field has been renamed in the base module
@mdietrichc2c @yankinmax
Remove self.pool + add correct attachment name
38918ab
@mdietrichc2c @yankinmax
View changed in batch_stock_picking
ca629ee
@guewen @yankinmax
Fix condition
95bf2ec
@guewen @yankinmax
Thread label generation to make it faster
1ec106f 
Port of OCA#27
@mdietrichc2c @yankinmax
Fix onchange for batch picking
574899e
@mdietrichc2c @yankinmax
Correct error treatment
80485a4
@guewen @yankinmax
Fix empty pdf issue
66b48b4 
Probably due to usage of multiple environment and cache. datas can't be
read from shipping.label but it can be read from ir.attachment
@mdietrichc2c @yankinmax
Add pick/pack info to all exceptions
55cfda3
@mdietrichc2c @yankinmax
e.message deprecated from 2.6
3920990
@guewen @yankinmax
Allow to print labels on operations without product
5e47465
oca-transbot and others added 23 commits February 24, 2025 10:39
@oca-transbot @yankinmax
Update translation files
663b6d9 
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
170c51e
@kv1612 @yankinmax
[MIG] delivery_carrier_label_batch: Migration to 14.0
fec0024
@oca-travis @yankinmax
[UPD] Update delivery_carrier_label_batch.pot
de58da4
@OCA-git-bot @yankinmax
[UPD] README.rst
cd57c45
@OCA-git-bot @yankinmax
[IMP] update dotfiles [ci skip]
0532784
@StephaneMangin @yankinmax
FIX delivery_carrier_label_batch invalid field
8ee6dda
@grindtildeath @yankinmax
delivery_carrier_label_batch: Allow testing labels generation
e26cdca 
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
aace022 
Co-authored-by: Hai Lang <hailn@trobz.com>
@sebalix @yankinmax
delivery_carrier_label_batch: empty cache once threads finished
e1b284c
@OCA-git-bot @yankinmax
delivery_carrier_label_batch 14.0.1.0.1
545b614
@OCA-git-bot @yankinmax
delivery_carrier_label_batch 14.0.1.0.2
b671cce
@StephaneMangin @yankinmax
[IMP] delivery_carrier_label_batch: tracking reference purge
59bc3e4 
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
1f1a34e
@OCA-git-bot @yankinmax
[UPD] README.rst
316cbf6
@weblate @yankinmax
Update translation files
2289f5e 
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
83da23b
@OCA-git-bot @yankinmax
[BOT] post-merge updates
b5cba53
@Ivorra78 @yankinmax
Added translation using Weblate (Spanish)
f8842b6
@Ivorra78 @yankinmax
Translated using Weblate (Spanish)
60b92c5 
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)
80b763b 
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)
a9f638c
@yankinmax
[IMP] delivery_carrier_label_batch: pre-commit auto fixes
ce2599a
@yankinmax yankinmax force-pushed the 18.0-mig-delivery_carrier_label_batch branch 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?

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.