[16.0][ADD] stock_lot_remove: Remove quants with a past removal date #2021
Conversation
lmignon
force-pushed
the
16.0-add-stock_expired_lot_move-lmi
branch
2 times, most recently
from
bbf6dbe to
6901647
Compare
|
ping @rousseldenis @jbaudoux |
lmignon
force-pushed
the
16.0-add-stock_expired_lot_move-lmi
branch
3 times, most recently
from
1a245e6 to
ae5fed1
Compare
rousseldenis
left a comment
rousseldenis
left a comment
There was a problem hiding this comment.
Seems great. Some remarks
| return [ | ||
| ("usage", "=", "internal"), | ||
| ("warehouse_id", "=", self.warehouse_id.id), | ||
| ("location_id", "child_of", self.warehouse_id.lar_orig_location_id.id), |
There was a problem hiding this comment.
Why not using the picking type to determine source and destination (default_location_src_id and default_location_dest_id fields) as this is the default Odoo behavior ?
There was a problem hiding this comment.
The source location will become a M2M to allows a fine grained selection of the locations to consider
|
Note: When moves are in packages not already done, we shouldn't do the process. |
| default=lambda self: self.env.context.get("active_model") == "stock.warehouse" | ||
| and self.env.context.get("active_id", False), |
There was a problem hiding this comment.
I think you have an active warehouse on user when you are in multi-wh. Would be nice to respect that
| removal_date = fields.Date( | ||
| string="Expiration Date", |
lmignon
force-pushed
the
16.0-add-stock_expired_lot_move-lmi
branch
from
ae5fed1 to
26f7184
Compare
lmignon
changed the title
lmignon
force-pushed
the
16.0-add-stock_expired_lot_move-lmi
branch
from
26f7184 to
475986e
Compare
I now exclude quants into packages. |
lmignon
force-pushed
the
16.0-add-stock_expired_lot_move-lmi
branch
3 times, most recently
from
a5a473e to
25859f2
Compare
lmignon
force-pushed
the
16.0-add-stock_expired_lot_move-lmi
branch
2 times, most recently
from
9b796d3 to
b307dff
Compare
lmignon
force-pushed
the
16.0-add-stock_expired_lot_move-lmi
branch
from
b307dff to
8f7bc92
Compare
3 participants
Automatically move remaining quants with a past removal date out of stocks