From 7a6efb900e190069a3d49c8c92068cb93c0c011c Mon Sep 17 00:00:00 2001 From: Pratik Date: Tue, 1 Apr 2025 17:26:46 +0530 Subject: [PATCH 01/12] fix: add mandatory fields --- .../doctype/crm_call_log/crm_call_log.json | 14 ++- crm/fcrm/doctype/fcrm_note/fcrm_note.json | 5 +- .../src/components/Modals/CallLogModal.vue | 50 +++----- frontend/src/components/Modals/NoteModal.vue | 74 +++++------ .../components/Modals/OrganizationModal.vue | 42 +++---- frontend/src/components/Modals/TaskModal.vue | 117 +++++++----------- 6 files changed, 123 insertions(+), 179 deletions(-) diff --git a/crm/fcrm/doctype/crm_call_log/crm_call_log.json b/crm/fcrm/doctype/crm_call_log/crm_call_log.json index cb7f91c2..cecd46c6 100644 --- a/crm/fcrm/doctype/crm_call_log/crm_call_log.json +++ b/crm/fcrm/doctype/crm_call_log/crm_call_log.json @@ -41,13 +41,15 @@ "fieldname": "from", "fieldtype": "Data", "in_list_view": 1, - "label": "From" + "label": "From", + "reqd": 1 }, { "fieldname": "status", "fieldtype": "Select", "label": "Status", - "options": "Initiated\nRinging\nIn Progress\nCompleted\nFailed\nBusy\nNo Answer\nQueued\nCanceled" + "options": "Initiated\nRinging\nIn Progress\nCompleted\nFailed\nBusy\nNo Answer\nQueued\nCanceled", + "reqd": 1 }, { "fieldname": "start_time", @@ -69,13 +71,15 @@ "in_list_view": 1, "in_standard_filter": 1, "label": "Type", - "options": "Incoming\nOutgoing" + "options": "Incoming\nOutgoing", + "reqd": 1 }, { "fieldname": "to", "fieldtype": "Data", "in_list_view": 1, - "label": "To" + "label": "To", + "reqd": 1 }, { "description": "Call duration in seconds", @@ -153,7 +157,7 @@ ], "index_web_pages_for_search": 1, "links": [], - "modified": "2025-01-22 17:57:59.289548", + "modified": "2025-04-01 16:01:54.479309", "modified_by": "Administrator", "module": "FCRM", "name": "CRM Call Log", diff --git a/crm/fcrm/doctype/fcrm_note/fcrm_note.json b/crm/fcrm/doctype/fcrm_note/fcrm_note.json index fc458fec..02477326 100644 --- a/crm/fcrm/doctype/fcrm_note/fcrm_note.json +++ b/crm/fcrm/doctype/fcrm_note/fcrm_note.json @@ -19,7 +19,8 @@ "fieldtype": "Data", "in_list_view": 1, "in_standard_filter": 1, - "label": "Title" + "label": "Title", + "reqd": 1 }, { "fieldname": "content", @@ -49,7 +50,7 @@ "link_fieldname": "note" } ], - "modified": "2024-01-19 21:56:30.123334", + "modified": "2025-04-01 15:30:14.742001", "modified_by": "Administrator", "module": "FCRM", "name": "FCRM Note", diff --git a/frontend/src/components/Modals/CallLogModal.vue b/frontend/src/components/Modals/CallLogModal.vue index b578f679..fbd0000f 100644 --- a/frontend/src/components/Modals/CallLogModal.vue +++ b/frontend/src/components/Modals/CallLogModal.vue @@ -1,55 +1,36 @@ From f0bf6962e7b0f5d3932fa26370be94b134a37724 Mon Sep 17 00:00:00 2001 From: Shariq Ansari Date: Mon, 7 Apr 2025 14:07:41 +0530 Subject: [PATCH 07/12] fix: do not show dropdown if only one option --- frontend/src/components/MultiActionButton.vue | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/frontend/src/components/MultiActionButton.vue b/frontend/src/components/MultiActionButton.vue index b7e5d3ce..02445aae 100644 --- a/frontend/src/components/MultiActionButton.vue +++ b/frontend/src/components/MultiActionButton.vue @@ -2,10 +2,13 @@
1) const activeButton = ref(props.options?.[0] || {}) const parsedOptions = computed(() => { From 40370067b20c92f339837745312b0594e7cc5b45 Mon Sep 17 00:00:00 2001 From: Shariq Ansari Date: Mon, 7 Apr 2025 14:13:55 +0530 Subject: [PATCH 08/12] fix: dynamic variant --- frontend/src/components/MultiActionButton.vue | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/frontend/src/components/MultiActionButton.vue b/frontend/src/components/MultiActionButton.vue index 02445aae..5319c1e3 100644 --- a/frontend/src/components/MultiActionButton.vue +++ b/frontend/src/components/MultiActionButton.vue @@ -1,7 +1,7 @@ From 469a22ef5fe6f6f7d302d4e42004e62968433764 Mon Sep 17 00:00:00 2001 From: Pratik Date: Mon, 7 Apr 2025 16:37:19 +0530 Subject: [PATCH 12/12] fix: default "assigned to" in deals and leads list view --- frontend/src/pages/Deals.vue | 3 --- frontend/src/pages/Leads.vue | 3 --- 2 files changed, 6 deletions(-) diff --git a/frontend/src/pages/Deals.vue b/frontend/src/pages/Deals.vue index 73e7f62c..093acddf 100644 --- a/frontend/src/pages/Deals.vue +++ b/frontend/src/pages/Deals.vue @@ -457,9 +457,6 @@ function parseRows(rows, columns = []) { } } else if (row == '_assign') { let assignees = JSON.parse(deal._assign || '[]') - if (!assignees.length && deal.deal_owner) { - assignees = [deal.deal_owner] - } _rows[row] = assignees.map((user) => ({ name: user, image: getUser(user).user_image, diff --git a/frontend/src/pages/Leads.vue b/frontend/src/pages/Leads.vue index 9d65cbd5..3956fb8e 100644 --- a/frontend/src/pages/Leads.vue +++ b/frontend/src/pages/Leads.vue @@ -480,9 +480,6 @@ function parseRows(rows, columns = []) { } } else if (row == '_assign') { let assignees = JSON.parse(lead._assign || '[]') - if (!assignees.length && lead.lead_owner) { - assignees = [lead.lead_owner] - } _rows[row] = assignees.map((user) => ({ name: user, image: getUser(user).user_image,