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

[16.0][IMP] stock_release_channel_shipment_advice_deliver: Improve wizard #935

Open
wants to merge 33 commits into
base: 16.0
Choose a base branch
from

Conversation

rousseldenis
Copy link
Contributor

@rousseldenis rousseldenis commented Oct 10, 2024

Improve wizard view:

image

@rousseldenis
Copy link
Contributor Author

@jbaudoux @lmignon

@rousseldenis rousseldenis added this to the 16.0 milestone Oct 10, 2024
@lmignon lmignon force-pushed the 16.0-imp-channel-deliver-message-dro branch from 289f3bf to d5c7362 Compare October 11, 2024 07:49
@rousseldenis
Copy link
Contributor Author

@jbaudoux @lmignon @phschmidt

Copy link
Contributor

@lmignon lmignon left a comment

Choose a reason for hiding this comment

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

LGTM

sbejaoui and others added 24 commits October 16, 2024 11:02
As the postpone operation will mark the current line as postponed,
we should iterate to the next line twice in the tests data to retrieve
the next line showed to the operator.
If there are several move lines for the same product,
e.g.: a product stored in several locations, and related to
the same stock move, when the first proposed line is postponed and when the
operator validates the second line, shopfloor will try to validate the stock move
and both lines. This is not correct as the qty_done field still contains the reserved
quantity.

So, when postponing a line, reset the qty_done field to 0.0 in order not to embed
that quantity in the validated picking.
Makes the  method into the cluster picking scenario an instance method in place of a staticmethod to ease override
With this change, in the cluster picking scenario, you can now provide a custom code on a stock device that will be used to order the operation when this device is used.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: wms-16.0/wms-16.0-shopfloor_batch_automatic_creation
Translate-URL: https://translation.odoo-community.org/projects/wms-16-0/wms-16-0-shopfloor_batch_automatic_creation/
Define open_picking_ids as picking_ids is growing over time
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: wms-16.0/wms-16.0-stock_release_channel
Translate-URL: https://translation.odoo-community.org/projects/wms-16-0/wms-16-0-stock_release_channel/
Currently translated at 100.0% (18 of 18 strings)

Translation: wms-16.0/wms-16.0-shopfloor_batch_automatic_creation
Translate-URL: https://translation.odoo-community.org/projects/wms-16-0/wms-16-0-shopfloor_batch_automatic_creation/it/
Currently translated at 100.0% (146 of 146 strings)

Translation: wms-16.0/wms-16.0-stock_release_channel
Translate-URL: https://translation.odoo-community.org/projects/wms-16-0/wms-16-0-stock_release_channel/it/
santostelmo and others added 9 commits October 16, 2024 11:02
…pt to initialize field with default values

The upgrade script should initialize the value with the sql default instead updating every existing
record (maybe billions).
Currently translated at 100.0% (42 of 42 strings)

Translation: wms-16.0/wms-16.0-stock_release_channel_shipment_advice_deliver
Translate-URL: https://translation.odoo-community.org/projects/wms-16-0/wms-16-0-stock_release_channel_shipment_advice_deliver/it/
…y pickings in confirm wizard

In order to let user aware of remaining pickings when delivering,
display them in the delivering wizard
@rousseldenis rousseldenis force-pushed the 16.0-imp-channel-deliver-message-dro branch from 26f5b85 to 132812f Compare October 16, 2024 09:02
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.

Something wrong with the diff. Can you properly rebase ?

/>
Are you sure you want to proceed with the delivery?
There are some preparations that have not been completed.<br />
If you choose to proceed, these preparations will be unreleased and won't be delivered or delivered partially.<br
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
If you choose to proceed, these preparations will be unreleased and won't be delivered or delivered partially.<br
If you choose to proceed, these deliveries will be unreleased and won't be delivered or delivered partially.<br

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.

8 participants