-
-
Notifications
You must be signed in to change notification settings - Fork 126
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
[14.0][IMP] shipment_advice: Autovalidate #128
[14.0][IMP] shipment_advice: Autovalidate #128
Conversation
vnikolayev1
commented
Jun 21, 2024
- modified shipment_advice, so whenever we set stock move to done, and if autovalidate shipment advice is enabled in company settings - we set shipment advice to done
- updated tests, covered backorder.
Sorry guys, messed up with rebase on #126 |
11fdb13
to
ca02118
Compare
This PR depends on #123 |
@@ -12,6 +12,7 @@ class StockMove(models.Model): | |||
ondelete="set null", | |||
string="Planned shipment", | |||
index=True, | |||
copy=False, |
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 needs to reverted when #123 is merged.
Could you please change the PR title to "[14.0][IMP] shipment_advice: Autovalidate" |
ca02118
to
dbe36fd
Compare
- modified shipment_advice, so whenever we set stock move to done, and if autovalidate shipment advice is enabled in company settings - we set shipment advice to done - updated tests.
dbe36fd
to
e03b20e
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 (Technical review)