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.

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

[18.0][MIG] stock_picking_delivery_link #987

Merged
merged 39 commits into from
Feb 26, 2025

Conversation

sebalix
Copy link
Contributor

@sebalix sebalix commented Feb 19, 2025

No description provided.

sebalix and others added 30 commits February 19, 2025 12:33
Adds link to the delivery on all intermediate operations (pick and pack
with a pick/pack/ship setup).
Currently translated at 100.0% (6 of 6 strings)

Translation: delivery-carrier-16.0/delivery-carrier-16.0-stock_picking_delivery_link
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-stock_picking_delivery_link/es/
Avoid test errors if the purchase module is installed

TT46563
…ge_type override

When the method _set_delivery_package_type is called from _pre_put_in_pack_hook in the stock.picking in the Odoo delivery module with the parameter batch_pack this error is raised

Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/odoo/http.py", line 1633, in _serve_db
    return service_model.retrying(self._serve_ir_http, self.env)
  File "/usr/lib/python3/dist-packages/odoo/service/model.py", line 133, in retrying
    result = func()
  File "/usr/lib/python3/dist-packages/odoo/http.py", line 1660, in _serve_ir_http
    response = self.dispatcher.dispatch(rule.endpoint, args)
  File "/usr/lib/python3/dist-packages/odoo/http.py", line 1864, in dispatch
    result = self.request.registry['ir.http']._dispatch(endpoint)
  File "/usr/lib/python3/dist-packages/odoo/addons/base/models/ir_http.py", line 154, in _dispatch
    result = endpoint(**request.params)
  File "/usr/lib/python3/dist-packages/odoo/http.py", line 697, in route_wrapper
    result = endpoint(self, *args, **params_ok)
  File "/usr/lib/python3/dist-packages/odoo/addons/web/controllers/dataset.py", line 46, in call_button
    action = self._call_kw(model, method, args, kwargs)
  File "/usr/lib/python3/dist-packages/odoo/addons/web/controllers/dataset.py", line 33, in _call_kw
    return call_kw(request.env[model], method, args, kwargs)
  File "/usr/lib/python3/dist-packages/odoo/api.py", line 468, in call_kw
    result = _call_kw_multi(method, model, args, kwargs)
  File "/usr/lib/python3/dist-packages/odoo/api.py", line 453, in _call_kw_multi
    result = method(recs, *args, **kwargs)
  File "/usr/lib/python3/dist-packages/odoo/addons/stock/models/stock_picking.py", line 1536, in action_put_in_pack
    res = self._pre_put_in_pack_hook(move_line_ids)
  File "/mnt/extra-addons/delivery-carrier/stock_picking_delivery_link/models/stock_picking.py", line 48, in _pre_put_in_pack_hook
    res = super()._pre_put_in_pack_hook(move_line_ids)
  File "/usr/lib/python3/dist-packages/odoo/addons/delivery/models/stock_picking.py", line 203, in _pre_put_in_pack_hook
    return self._set_delivery_package_type(batch_pack=len(move_line_ids.picking_id) > 1)
TypeError: _set_delivery_package_type() got an unexpected keyword argument 'batch_pack'
Currently translated at 100.0% (6 of 6 strings)

Translation: delivery-carrier-16.0/delivery-carrier-16.0-stock_picking_delivery_link
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-stock_picking_delivery_link/it/
Avoid test errors if the purchase module is installed

Related to OCA@9924c78
Similar to OCA@3cce9bb

TT46563
OCA-git-bot and others added 7 commits February 19, 2025 12:33
Currently translated at 100.0% (6 of 6 strings)

Translation: delivery-carrier-16.0/delivery-carrier-16.0-stock_picking_delivery_link
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-stock_picking_delivery_link/it/
Currently translated at 83.3% (5 of 6 strings)

Translation: delivery-carrier-16.0/delivery-carrier-16.0-stock_picking_delivery_link
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-stock_picking_delivery_link/fr/
Currently translated at 100.0% (6 of 6 strings)

Translation: delivery-carrier-16.0/delivery-carrier-16.0-stock_picking_delivery_link
Translate-URL: https://translation.odoo-community.org/projects/delivery-carrier-16-0/delivery-carrier-16-0-stock_picking_delivery_link/tr/
@sebalix
Copy link
Contributor Author

sebalix commented Feb 19, 2025

/ocabot migration stock_picking_delivery_link

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Feb 19, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Feb 19, 2025
23 tasks
@sebalix sebalix force-pushed the 18-mig-stock_picking_delivery_link branch 2 times, most recently from 16e2149 to a6a176d Compare February 19, 2025 12:56
@sebalix sebalix marked this pull request as ready for review February 19, 2025 12:59
Copy link
Contributor

@ajaniszewska-dev ajaniszewska-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG

Copy link
Contributor

@jbaudoux jbaudoux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

carrier_id field is now in stock_delivery module

@sebalix sebalix force-pushed the 18-mig-stock_picking_delivery_link branch from a6a176d to a3fa0a9 Compare February 26, 2025 10:57
@sebalix
Copy link
Contributor Author

sebalix commented Feb 26, 2025

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-987-by-sebalix-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 7f95557 into OCA:18.0 Feb 26, 2025
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.