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

15.0 stock picking on hold #2608

Open
wants to merge 8 commits into
base: 15.0
Choose a base branch
from

Conversation

alihashaam
Copy link

No description provided.

@alihashaam
Copy link
Author

CI is failing because of another module "sale_order_discount_invoicing", for which tests are failing

@rousseldenis
Copy link
Sponsor Contributor

@alihashaam Thanks for this.

IMHO, this has same functionality as https://github.com/OCA/sale-workflow/tree/15.0/sale_stock_picking_blocking

As done in v16, the blocking reason is now computed stored. So, your module could depends on invoice_status field, then, assign the blocking reason.

https://github.com/OCA/sale-workflow/blob/16.0/sale_stock_picking_blocking/models/sale_order.py#L33

What do you think ?

@rousseldenis rousseldenis added this to the 15.0 milestone Jul 20, 2023
@ortlam
Copy link
Member

ortlam commented Sep 4, 2023

@rousseldenis Thank you for the suggestion, it makes sense to base this module on sale_stock_picking_blocking.

@jans23
Copy link

jans23 commented Sep 6, 2023

I don't think that test failure is related to this module stock_picking_on_hold.

[15.0][upd] conditional readonly added to hold_picking_until_payment

[15.0][upd] hold_picking_until_payment set to true by default

[15.0][IMP] Base stock_picking_on_hold on sale_stock_picking_blocking
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants