-
-
Notifications
You must be signed in to change notification settings - Fork 486
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
[16.0][ADD] mrp_stock_owner_restriction #1013
base: 16.0
Are you sure you want to change the base?
[16.0][ADD] mrp_stock_owner_restriction #1013
Conversation
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. |
@AungKoKoLin1997 Can you please rebase and push? |
3637991
to
71ef736
Compare
mrp_stock_owner_restriction/tests/test_mrp_stock_owner_restriction.py
Outdated
Show resolved
Hide resolved
mrp_stock_owner_restriction/tests/test_mrp_stock_owner_restriction.py
Outdated
Show resolved
Hide resolved
mrp_stock_owner_restriction/tests/test_mrp_stock_owner_restriction.py
Outdated
Show resolved
Hide resolved
mrp_stock_owner_restriction/tests/test_mrp_stock_owner_restriction.py
Outdated
Show resolved
Hide resolved
mrp_stock_owner_restriction/tests/test_mrp_stock_owner_restriction.py
Outdated
Show resolved
Hide resolved
e877e92
to
9303ccd
Compare
@@ -0,0 +1,2 @@ | |||
This module adopts the functionality of stock_owner_restriction in the manufacturing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This module adopts the functionality of stock_owner_restriction in the manufacturing | |
This module adapts the functionality of stock_owner_restriction in the manufacturing |
9303ccd
to
5753819
Compare
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. |
f525d3e
to
6ddd377
Compare
6ddd377
to
a9e376b
Compare
a9e376b
to
7c8bf85
Compare
/ocabot rebase Would love a new runboat for func testing. |
Hi @bosd. Your command failed:
Ocabot commands
More information
|
7c8bf85
to
348f537
Compare
@bosd I rerun the CI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we forgot to use the owner_restriction
field defined in the production.
owner_restriction = self.picking_type_id.owner_restriction | ||
if owner_restriction in ("unassigned_owner", "picking_partner"): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
owner_restriction = self.picking_type_id.owner_restriction | |
if owner_restriction in ("unassigned_owner", "picking_partner"): | |
if production.owner_restriction in ("unassigned_owner", "picking_partner"): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
# owner_restriction = fields.Selection(related="picking_type_id.owner_restriction") | ||
|
||
def action_validate(self): | ||
owner_restriction = self.mo_id.picking_type_id.owner_restriction |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
owner_restriction = self.mo_id.picking_type_id.owner_restriction | |
owner_restriction = self.mo_id.owner_restriction |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
self.ensure_one() | ||
mo_to_unbuild = self._get_mo_to_unbuild() | ||
if mo_to_unbuild: | ||
return mo_to_unbuild.picking_type_id.owner_restriction |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return mo_to_unbuild.picking_type_id.owner_restriction | |
return mo_to_unbuild.owner_restriction |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
348f537
to
2f92a5d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎇
This module adopts the functionality of stock_owner_restriction in the manufacturing order. This module will assign the owner for the produced product.
Related - OCA/stock-logistics-workflow#1231
@qrtl