-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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: provision to have limited parameters for in-process quality inspection #28219
feat: provision to have limited parameters for in-process quality inspection #28219
Conversation
5f02ab3
to
5da83d2
Compare
5da83d2
to
6c45f8b
Compare
for d in parameters: | ||
if reading.specification == d.specification: | ||
reading.update(d) | ||
reading.status = "Accepted" |
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.
Why status as "Accepted", it's depending on the condition
@@ -292,6 +293,10 @@ def update_sub_operation_status(self): | |||
row.completed_time = 0.0 | |||
row.completed_qty = 0.0 | |||
|
|||
def set_quality_inspection_template(self): | |||
qi_template = frappe.db.get_value('Operation', self.operation, 'quality_inspection_template') | |||
self.quality_inspection_template = qi_template |
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.
Use fetch from property in the doctype
@Mergifyio backport version-13-pre-release |
✅ Backports have been created
|
…se/pr-28219 feat: provision to have limited parameters for in-process quality inspection (backport #28219)
@Mergifyio backport develop |
✅ Backports have been created
|
…-release/pr-28219 feat: provision to have limited parameters for in-process quality inspection (backport frappe#28219)
Additions:
Screenshots/Gifs:
Documentation:
https://docs.erpnext.com/docs/v13/user/manual/en/manufacturing/operation