-
-
Notifications
You must be signed in to change notification settings - Fork 63
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
Add module logistics_requisition_donation #97
Add module logistics_requisition_donation #97
Conversation
66a8278
to
a4f1707
Compare
Tell me if I can help with the red build @yvaucher. |
I edited my last comment because I messed up with another PR's travis build. |
a4f1707
to
4a36b30
Compare
Rebased on top of #92 |
7f8cf60
to
b0931b2
Compare
|
||
@api.onchange('procurement_method') | ||
def onchange_source_type_warning(self): | ||
if self.procurement_method: |
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.
To fix
not in ('wh_dispatch', 'other')
b0931b2
to
cb78ae2
Compare
cb78ae2
to
c5269e9
Compare
c5269e9
to
bb62a9f
Compare
👍 LGTM |
:params line: browse record of origin requistion.line | ||
:params agreement: browse record of origin agreement | ||
:params qty: quantity to be set on source line | ||
:param self: record set of origin requistion.line |
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.
A detail, but I wouldn't document things that are not specific to this method like self
.
|
||
if not agreement.product_id.id == self.product_id.id: |
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.
Maybe the comparison would work on the records without .id
?
Logic is moved on logistic_requisition module and method are simplified with a bit of API v8.0 This change intends to allow to inherit from auto creation of source without having to depends on framework_agreement_sourcing
bb62a9f
to
7cc3635
Compare
@lepistone fixed few things according to your remarks |
Great! I'm merging when green. |
…nation Add module logistics_requisition_donation
This one is based on (includes other PR commits)