-
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
[tests] update travis.yml to run ui-tests #9748
Conversation
d = frappe.get_doc("Item Group", d.name) | ||
if d.route: | ||
clear_cache(d.route) | ||
item_group_name = frappe.db.get_value("Item Group", d.get('name')) |
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.
the fix already sent by Makarand and it is merged
daily_work_summary = frappe.get_doc('Daily Work Summary', | ||
frappe.get_all('Daily Work Summary')[0].name) | ||
|
||
summary = daily_work_summary.get_summary_message() | ||
|
||
print summary |
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.
please remove print
statement
Pull Request SummaryImage or animted GIF Not AddedPlease add an image or animated GIF as proof that you have manually tested this contribution. Hint: use LiceCAP to capture animated GIFs. Test Case Not Added / UpdatedSince you have changed a Python file, you must update the relevant python test case. If there is no test coverage for this code, then please add it. Documentation not updatedSince you have changed a DocType, check if this impacts any of the documentation related to it. Result
This summary was automatically generated based on this script |
@nabinhait @mbauskar pls don't update this PR. Still working on it. |
No description provided.