-
Notifications
You must be signed in to change notification settings - Fork 24
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
feat: added customer and supplier information to Batch #1187
feat: added customer and supplier information to Batch #1187
Conversation
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 have shared feedback with @rvpasari
@@ -134,9 +135,8 @@ | |||
}, | |||
{ | |||
"fieldname": "supplier", | |||
"fieldtype": "Link", | |||
"fieldtype": "Data", |
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.
@MyuddinKhatri did you change Supplier to a data field ?
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.
Yes
@@ -100,6 +100,7 @@ def on_submit(self): | |||
self.update_so_in_serial_number() | |||
self.update_package_tag() | |||
self.update_package_tag_is_used() | |||
self.update_batch_with_customer_providing_item() |
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.
update_batch_with_customer_provided_item
@@ -689,6 +690,16 @@ def update_stock_ledger(self): | |||
|
|||
self.make_sl_entries(sl_entries, self.amended_from and 'Yes' or 'No') | |||
|
|||
def update_batch_with_customer_providing_item(self): |
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.
update_batch_with_customer_provided_item
Ref: Asana