[14.0][ADD] shopfloor_full_location_reservation: Full location reservation for shopfloor#792
Conversation
|
cc @jbaudoux |
|
Tests are missing |
| _inherit = "shopfloor.zone.picking" | ||
|
|
||
| def _handle_full_location_package_reservation(self, package): | ||
| return package and self.work.menu.full_location_reservation |
There was a problem hiding this comment.
It is not a full location reservation, it is a full package reservation.
If you had scanned a product not part of a package, it should trigger a full product reservation. Same for lot.
| full_location_reservation_is_possible = fields.Boolean( | ||
| compute="_compute_full_location_reservation_is_possible" | ||
| ) | ||
| full_location_reservation = fields.Boolean( |
There was a problem hiding this comment.
Shouldn't this be replaced by full_source_reservation ?
|
This was initially developed for the |
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
github-actions
Bot
added
the
stale
jbaudoux
added
no stale
mt-software-de
force-pushed
the
14-full_location_reservation_shopfloor
branch
from
31b23c6 to
8e8ca98
Compare
|
@jbaudoux @rousseldenis This is now used quite some time in production. How should we move forward with this? |
mt-software-de
force-pushed
the
14-full_location_reservation_shopfloor
branch
from
684d9ec to
cac81b0
Compare
…ion on move but on move line As a stock move can point to several move lines on several stock locations, don't do the full reservation process on move but on lines.
mt-software-de
force-pushed
the
14-full_location_reservation_shopfloor
branch
from
cac81b0 to
5509f77
Compare
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
It looks like something changed on |
|
Congratulations, your PR was merged at fbde718. Thanks a lot for contributing to OCA. ❤️ |
4 participants
No description provided.