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

[14.0][ADD] delivery_line_qty_delivered #576

Conversation

mt-software-de
Copy link
Contributor

@mt-software-de mt-software-de commented Dec 11, 2022

This modules changes the Delivered Quantity (qty_delivered) of a delivery sale.order.line
if one other sale.order.line is delivered
the qty_delivered of the delivery line is changed to its Quantity (product_uom_qty)

cc @jbaudoux

@mt-software-de mt-software-de force-pushed the 14.0_add_delivery_line_qty_delivered branch 2 times, most recently from c6ecf45 to d457042 Compare December 12, 2022 05:41
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.

The dependencies seems to cause a non deterministic result

delivery_line_qty_delivered/models/sale_order_line.py Outdated Show resolved Hide resolved
delivery_line_qty_delivered/models/sale_order.py Outdated Show resolved Hide resolved
@mt-software-de mt-software-de force-pushed the 14.0_add_delivery_line_qty_delivered branch 3 times, most recently from 8b2f92a to 2e57b42 Compare December 12, 2022 10:09
@rousseldenis rousseldenis added this to the 14.0 milestone Jan 23, 2023
@mt-software-de
Copy link
Contributor Author

The dependencies seems to cause a non deterministic result

I removed the dependency.
@jbaudoux or @rousseldenis could you maybe have a look at tests?
Because the tests are running locally.

@mt-software-de mt-software-de force-pushed the 14.0_add_delivery_line_qty_delivered branch from 97cc9c6 to db17f17 Compare April 25, 2023 10:15
@rousseldenis
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 14.0.

@OCA-git-bot OCA-git-bot force-pushed the 14.0_add_delivery_line_qty_delivered branch from db17f17 to 70bf70f Compare July 25, 2023 07:19
@rousseldenis
Copy link
Contributor

@mt-software-de I would try to install all the modules installed before this one (see log details) and see which one is modifying standard behavior.

@mt-software-de
Copy link
Contributor Author

@mt-software-de I would try to install all the modules installed before this one (see log details) and see which one is modifying standard behavior.

Thxs, Unfortunately I have to do that.

@mt-software-de mt-software-de force-pushed the 14.0_add_delivery_line_qty_delivered branch from 70bf70f to 91d0b50 Compare October 10, 2023 20:43
Changes the Delivered Quantity (qty_delivered) of a delivery sale.order.line
if one other sale.order.line is delivered the qty_deliverd of the delvivery line
is changed to its Quantity (product_uom_qty)
@mt-software-de mt-software-de force-pushed the 14.0_add_delivery_line_qty_delivered branch from 91d0b50 to c279fa6 Compare October 10, 2023 20:50
@mt-software-de
Copy link
Contributor Author

This was now moved to OCA/sale-workflow#2717

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.

4 participants