From 29d4ae671629adc295e9b1a1ee98df0387c2f221 Mon Sep 17 00:00:00 2001 From: Himanshu Date: Mon, 20 May 2019 15:14:07 +0530 Subject: [PATCH] feat(Communication): Move Timeline DocType and Name to Dynamic Links (#17593) * fix: ambigious error --- erpnext/accounts/party.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/erpnext/accounts/party.py b/erpnext/accounts/party.py index f25473e3089..29bae4b609c 100644 --- a/erpnext/accounts/party.py +++ b/erpnext/accounts/party.py @@ -463,8 +463,8 @@ def get_timeline_data(doctype, name): after = add_years(None, -1).strftime('%Y-%m-%d') group_by='group by date(creation)' - data = get_communication_data(doctype, name, - fields=fields, after=after, group_by=group_by, as_dict=False) + data = get_communication_data(doctype, name, after=after, group_by='group by date(`tabCommunication`.creation)', + fields='date(`tabCommunication`.creation), count(`tabCommunication`.name)',as_dict=False) # fetch and append data from Activity Log data += frappe.db.sql("""select {fields}