[18.0][IMP] pos_sale_picking_keep: Add a strategy to keep sale pickings and/or POS pickings too#1545
[18.0][IMP] pos_sale_picking_keep: Add a strategy to keep sale pickings and/or POS pickings too#1545rousseldenis wants to merge 5 commits into
Conversation
|
Hi @pedrobaeza, |
OCA-git-bot
added
series:18.0
mod:pos_sale_picking_keep
pedrobaeza
left a comment
pedrobaeza
left a comment
There was a problem hiding this comment.
If you mix directly items from PoS from other loaded from sales, I think there's no discussion: both should be preserved, the ones from PoS in the PoS picking, and the sales one in the sales pickings, so I think there shouldn't be an option, just to proceed that way.
We were previously discarding everything from PoS, because we don't have that mixed option (for now), but if having it in the future, I don't conceive another option except that.
I wanted to keep your behaviour as I didn't know if you have particular processes on your side. So, I keep the option activation or not through a Boolean in place of Selection. |
| if all( | ||
| picking.state == "done" for picking in pos_lines.order_id.picking_ids | ||
| ): |
There was a problem hiding this comment.
not sure if we should prevent empty list (all([]) == True) here ?
| if all( | |
| picking.state == "done" for picking in pos_lines.order_id.picking_ids | |
| ): | |
| if pos_lines.order_id.picking_ids and all( | |
| picking.state == "done" for picking in pos_lines.order_id.picking_ids | |
| ): |
There was a problem hiding this comment.
@petrus-v In fact, no. We should wait : odoo/odoo#253333
|
@rousseldenis you can go directly with no possible choice :) |
I'd prefer not as using stuffs for tests in code is not my cup of tea ;-) |
|
I don't get you. What do you mean by |
You want to say, install this module and the behaviour changes. Correct ? |
|
Well, yes, so I get your point. Let's keep the activation by pos config. |
…t pickings As in some cases, you want to mix in POS orders the payment of an existing sale and added products in the same POS order, you still want to keep both pickings linked to sale orders AND pos orders. This introduces the keep picking strategy that allows to choose per POS configuration the strategy that fits the best to your case.
The behaviour should be the same when the configuration is applied for both POS pickings and sale orders one.
rousseldenis
force-pushed
the
18.0-imp-pos_sale_picking_keep-dro
branch
from
3438237 to
2d43560
Compare
4 participants
@victoralmau @pedrobaeza