-
-
Notifications
You must be signed in to change notification settings - Fork 787
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
[IMP] purchase_lot : lot_id is computed from stock moves so that it c… #2247
base: 16.0
Are you sure you want to change the base?
[IMP] purchase_lot : lot_id is computed from stock moves so that it c… #2247
Conversation
Hi @florian-dacosta, |
…an be updated after confirmation and purchase order lines aren't merged even if lots aren't defined at creation
3dd3663
to
2c0d84a
Compare
fc3a67a
to
926cafd
Compare
…update-restrict-lot_pol 16.0 imp purchase lot update restrict lot on purchase order line
926cafd
to
aa5228d
Compare
Co-authored-by: Kévin Roche <[email protected]>
Now that OCA/stock-logistics-workflow#1560 has been merged, can we process this one too ? Regards, |
@metaminux I don't really like this because the lot on the purchase line could change anytime silently, no matter the state of the purchase order. In most cases (or at least in some cases), we want the lot on the purchase so we can send it (+ eventually information stored on the lot) to the supplier, so that it can send a product accordingly. I would be ok to change the lot if the purchase is still at draft state, but otherwise, I'd rather not. |
You're right, maybe I'm too focused on my specific use case which is inter company workflow... I can only imagine 2 use cases here :
In first case, I suppose you are generating lots before confirming the sale order and you don't want to change it after (which is the behaviour we got for the moment) Now, in first case, if we change the lots after confirmation, it can be just switching between sales or you need to inform your supplier you want some new lots... Of course, I could be wrong about use cases here, so feel free to correct me. |
…an be updated after confirmation and purchase order lines aren't merged even if lots aren't defined at creation
As sale_order_lot_selection allows to change lot after confirmation, we think stock moves and related objects should propagate those changes (here purchase order lines)
This PR is linked to these others PRs :