From 2c3a660b99e7cc3a1443d50470b3b773d3a245fc Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Wed, 11 Jun 2025 13:00:02 +0530
Subject: [PATCH 01/11] fix: added deal value field

(cherry picked from commit 01f0213693d0885228b852bbf852fff254df988f)
---
 crm/fcrm/doctype/crm_deal/crm_deal.json | 25 +++++++++++++++++++++----
 1 file changed, 21 insertions(+), 4 deletions(-)

diff --git a/crm/fcrm/doctype/crm_deal/crm_deal.json b/crm/fcrm/doctype/crm_deal/crm_deal.json
index 67f441c3..10e56be4 100644
--- a/crm/fcrm/doctype/crm_deal/crm_deal.json
+++ b/crm/fcrm/doctype/crm_deal/crm_deal.json
@@ -11,11 +11,14 @@
   "naming_series",
   "organization",
   "next_step",
-  "probability",
   "column_break_ijan",
   "status",
-  "close_date",
   "deal_owner",
+  "section_break_jgpm",
+  "probability",
+  "deal_value",
+  "column_break_kpxa",
+  "close_date",
   "contacts_tab",
   "contacts",
   "contact",
@@ -91,7 +94,7 @@
   {
    "fieldname": "close_date",
    "fieldtype": "Date",
-   "label": "Close Date"
+   "label": "Expected Closure Date"
   },
   {
    "fieldname": "next_step",
@@ -374,12 +377,26 @@
    "label": "Net Total",
    "options": "currency",
    "read_only": 1
+  },
+  {
+   "fieldname": "section_break_jgpm",
+   "fieldtype": "Section Break"
+  },
+  {
+   "fieldname": "deal_value",
+   "fieldtype": "Currency",
+   "label": "Deal Value",
+   "options": "currency"
+  },
+  {
+   "fieldname": "column_break_kpxa",
+   "fieldtype": "Column Break"
   }
  ],
  "grid_page_length": 50,
  "index_web_pages_for_search": 1,
  "links": [],
- "modified": "2025-05-12 12:30:55.415282",
+ "modified": "2025-06-11 12:58:22.439045",
  "modified_by": "Administrator",
  "module": "FCRM",
  "name": "CRM Deal",

From 1c7ea0b44b8a4cafa0107fdddcb17c6268d17f5c Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Wed, 11 Jun 2025 13:01:22 +0530
Subject: [PATCH 02/11] fix: added probability field in deal status

(cherry picked from commit fdca27bb81533b45eac0ba9d45c2ca1d094d2fae)
---
 .../doctype/crm_deal_status/crm_deal_status.json   | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/crm/fcrm/doctype/crm_deal_status/crm_deal_status.json b/crm/fcrm/doctype/crm_deal_status/crm_deal_status.json
index b0374ca1..ae026c74 100644
--- a/crm/fcrm/doctype/crm_deal_status/crm_deal_status.json
+++ b/crm/fcrm/doctype/crm_deal_status/crm_deal_status.json
@@ -8,7 +8,8 @@
  "field_order": [
   "deal_status",
   "color",
-  "position"
+  "position",
+  "probability"
  ],
  "fields": [
   {
@@ -32,11 +33,17 @@
    "fieldtype": "Int",
    "in_list_view": 1,
    "label": "Position"
+  },
+  {
+   "fieldname": "probability",
+   "fieldtype": "Percent",
+   "label": "Probability"
   }
  ],
+ "grid_page_length": 50,
  "index_web_pages_for_search": 1,
  "links": [],
- "modified": "2024-01-19 21:56:44.552134",
+ "modified": "2025-06-11 13:00:34.518808",
  "modified_by": "Administrator",
  "module": "FCRM",
  "name": "CRM Deal Status",
@@ -68,7 +75,8 @@
    "write": 1
   }
  ],
+ "row_format": "Dynamic",
  "sort_field": "modified",
  "sort_order": "DESC",
  "states": []
-}
\ No newline at end of file
+}

From b0474e87644f7849055cbe300effcfc819c462a5 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Wed, 11 Jun 2025 14:18:25 +0530
Subject: [PATCH 03/11] fix: update and reset value in triggerOnChange method

(cherry picked from commit 7d79cbf5bdd664e76fb90b994d0d4f5c257c0ed0)
---
 frontend/src/components/Controls/Grid.vue     |  3 +--
 frontend/src/components/FieldLayout/Field.vue |  6 ++----
 frontend/src/components/SidePanelLayout.vue   |  4 +---
 frontend/src/data/document.js                 | 20 ++++++++++++-------
 4 files changed, 17 insertions(+), 16 deletions(-)

diff --git a/frontend/src/components/Controls/Grid.vue b/frontend/src/components/Controls/Grid.vue
index 54bc29dc..19240a86 100644
--- a/frontend/src/components/Controls/Grid.vue
+++ b/frontend/src/components/Controls/Grid.vue
@@ -509,8 +509,7 @@ const deleteRows = () => {
 }
 
 function fieldChange(value, field, row) {
-  row[field.fieldname] = value
-  triggerOnChange(field.fieldname, row)
+  triggerOnChange(field.fieldname, value, row)
 }
 
 function getDefaultValue(defaultValue, fieldtype) {
diff --git a/frontend/src/components/FieldLayout/Field.vue b/frontend/src/components/FieldLayout/Field.vue
index b517d973..07f530cd 100644
--- a/frontend/src/components/FieldLayout/Field.vue
+++ b/frontend/src/components/FieldLayout/Field.vue
@@ -332,12 +332,10 @@ const getPlaceholder = (field) => {
 }
 
 function fieldChange(value, df) {
-  data.value[df.fieldname] = value
-
   if (isGridRow) {
-    triggerOnChange(df.fieldname, data.value)
+    triggerOnChange(df.fieldname, value, data.value)
   } else {
-    triggerOnChange(df.fieldname)
+    triggerOnChange(df.fieldname, value)
   }
 }
 
diff --git a/frontend/src/components/SidePanelLayout.vue b/frontend/src/components/SidePanelLayout.vue
index 3fe8fd1a..b9512b46 100644
--- a/frontend/src/components/SidePanelLayout.vue
+++ b/frontend/src/components/SidePanelLayout.vue
@@ -489,9 +489,7 @@ function parsedField(field) {
 async function fieldChange(value, df) {
   if (props.preview) return
 
-  document.doc[df.fieldname] = value
-
-  await triggerOnChange(df.fieldname)
+  await triggerOnChange(df.fieldname, value)
 
   document.save.submit(null, {
     onSuccess: () => {
diff --git a/frontend/src/data/document.js b/frontend/src/data/document.js
index 7d456676..76e35a9e 100644
--- a/frontend/src/data/document.js
+++ b/frontend/src/data/document.js
@@ -117,20 +117,26 @@ export function useDocument(doctype, docname) {
     await trigger(handler)
   }
 
-  async function triggerOnChange(fieldname, row) {
+  async function triggerOnChange(fieldname, value, row) {
+    const oldValue = documentsCache[doctype][docname || ''].doc[fieldname]
+    documentsCache[doctype][docname || ''].doc[fieldname] = value
+
     const handler = async function () {
+      this.value = value
+      this.oldValue = oldValue
       if (row) {
         this.currentRowIdx = row.idx
-        this.value = row[fieldname]
-        this.oldValue = getOldValue(fieldname, row)
-      } else {
-        this.value = documentsCache[doctype][docname || ''].doc[fieldname]
-        this.oldValue = getOldValue(fieldname)
       }
       await this[fieldname]?.()
     }
 
-    await trigger(handler, row)
+    try {
+      await trigger(handler, row)
+    } catch (error) {
+      documentsCache[doctype][docname || ''].doc[fieldname] = oldValue
+      console.error(handler)
+      throw error
+    }
   }
 
   async function triggerOnRowAdd(row) {

From 5fff00338a258472f5c86f5c6f09ca56cf2b6022 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Wed, 11 Jun 2025 14:19:16 +0530
Subject: [PATCH 04/11] fix: added throwError global method

(cherry picked from commit 9af300bba8a59cb49ecce26fe5e790fdb646cd99)
---
 frontend/src/data/script.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/frontend/src/data/script.js b/frontend/src/data/script.js
index b809f744..47185bab 100644
--- a/frontend/src/data/script.js
+++ b/frontend/src/data/script.js
@@ -46,6 +46,11 @@ export function getScript(doctype, view = 'Form') {
     helpers.router = router
     helpers.call = call
 
+    helpers.throwError = (message) => {
+      toast.error(message || __('An error occurred'))
+      throw new Error(message || __('An error occurred'))
+    }
+
     helpers.crm = {
       makePhoneCall: makeCall,
     }

From 87a38250f6549d1aef2c102a8323c823f36eedda Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Wed, 11 Jun 2025 14:20:11 +0530
Subject: [PATCH 05/11] fix: trigger on change on status change

(cherry picked from commit e9e0aa357b1b1596f57d8bda0037f36514972169)
---
 frontend/src/components/Modals/DealModal.vue |  4 ++--
 frontend/src/components/Modals/LeadModal.vue |  4 ++--
 frontend/src/pages/Deal.vue                  | 14 ++++++++++++--
 frontend/src/pages/Lead.vue                  | 15 ++++++++++-----
 frontend/src/pages/MobileDeal.vue            | 14 ++++++++++++--
 frontend/src/pages/MobileLead.vue            | 14 ++++++++++++--
 frontend/src/stores/statuses.js              | 13 +++++++++----
 7 files changed, 59 insertions(+), 19 deletions(-)

diff --git a/frontend/src/components/Modals/DealModal.vue b/frontend/src/components/Modals/DealModal.vue
index 1c516407..e4ce7419 100644
--- a/frontend/src/components/Modals/DealModal.vue
+++ b/frontend/src/components/Modals/DealModal.vue
@@ -94,7 +94,7 @@ const show = defineModel()
 const router = useRouter()
 const error = ref(null)
 
-const { document: deal } = useDocument('CRM Deal')
+const { document: deal, triggerOnChange } = useDocument('CRM Deal')
 
 const hasOrganizationSections = ref(true)
 const hasContactSections = ref(true)
@@ -164,7 +164,7 @@ const tabs = createResource({
 })
 
 const dealStatuses = computed(() => {
-  let statuses = statusOptions('deal')
+  let statuses = statusOptions('deal', null, [], triggerOnChange)
   if (!deal.doc.status) {
     deal.doc.status = statuses[0].value
   }
diff --git a/frontend/src/components/Modals/LeadModal.vue b/frontend/src/components/Modals/LeadModal.vue
index bf1461d1..1976ad3c 100644
--- a/frontend/src/components/Modals/LeadModal.vue
+++ b/frontend/src/components/Modals/LeadModal.vue
@@ -70,10 +70,10 @@ const router = useRouter()
 const error = ref(null)
 const isLeadCreating = ref(false)
 
-const { document: lead } = useDocument('CRM Lead')
+const { document: lead, triggerOnChange } = useDocument('CRM Lead')
 
 const leadStatuses = computed(() => {
-  let statuses = statusOptions('lead')
+  let statuses = statusOptions('lead', null, [], triggerOnChange)
   if (!lead.doc.status) {
     lead.doc.status = statuses?.[0]?.value
   }
diff --git a/frontend/src/pages/Deal.vue b/frontend/src/pages/Deal.vue
index e3d09fa7..34e06141 100644
--- a/frontend/src/pages/Deal.vue
+++ b/frontend/src/pages/Deal.vue
@@ -23,7 +23,14 @@
       />
       <Dropdown
         v-if="document.doc"
-        :options="statusOptions('deal', document, deal.data._customStatuses)"
+        :options="
+          statusOptions(
+            'deal',
+            document,
+            deal.data._customStatuses,
+            triggerOnChange,
+          )
+        "
       >
         <template #default="{ open }">
           <Button :label="document.doc.status">
@@ -726,7 +733,10 @@ function openEmailBox() {
   activities.value.emailBox.show = true
 }
 
-const { assignees, document } = useDocument('CRM Deal', props.dealId)
+const { assignees, document, triggerOnChange } = useDocument(
+  'CRM Deal',
+  props.dealId,
+)
 
 function reloadAssignees(data) {
   if (data?.hasOwnProperty('deal_owner')) {
diff --git a/frontend/src/pages/Lead.vue b/frontend/src/pages/Lead.vue
index d02d5af9..17612e19 100644
--- a/frontend/src/pages/Lead.vue
+++ b/frontend/src/pages/Lead.vue
@@ -23,7 +23,14 @@
       />
       <Dropdown
         v-if="document.doc"
-        :options="statusOptions('lead', document, lead.data._customStatuses)"
+        :options="
+          statusOptions(
+            'lead',
+            document,
+            lead.data._customStatuses,
+            triggerOnChange,
+          )
+        "
       >
         <template #default="{ open }">
           <Button :label="document.doc.status">
@@ -610,10 +617,8 @@ const existingOrganizationChecked = ref(false)
 const existingContact = ref('')
 const existingOrganization = ref('')
 
-const { triggerConvertToDeal, assignees, document } = useDocument(
-  'CRM Lead',
-  props.leadId,
-)
+const { triggerConvertToDeal, triggerOnChange, assignees, document } =
+  useDocument('CRM Lead', props.leadId)
 
 async function convertToDeal() {
   if (existingContactChecked.value && !existingContact.value) {
diff --git a/frontend/src/pages/MobileDeal.vue b/frontend/src/pages/MobileDeal.vue
index 71995d88..1920266f 100644
--- a/frontend/src/pages/MobileDeal.vue
+++ b/frontend/src/pages/MobileDeal.vue
@@ -11,7 +11,14 @@
       <div class="absolute right-0">
         <Dropdown
           v-if="document.doc"
-          :options="statusOptions('deal', document, deal.data._customStatuses)"
+          :options="
+            statusOptions(
+              'deal',
+              document,
+              deal.data._customStatuses,
+              triggerOnChange,
+            )
+          "
         >
           <template #default="{ open }">
             <Button :label="document.doc.status">
@@ -612,7 +619,10 @@ async function deleteDeal(name) {
   router.push({ name: 'Deals' })
 }
 
-const { assignees, document } = useDocument('CRM Deal', props.dealId)
+const { assignees, document, triggerOnChange } = useDocument(
+  'CRM Deal',
+  props.dealId,
+)
 
 function reloadAssignees(data) {
   if (data?.hasOwnProperty('deal_owner')) {
diff --git a/frontend/src/pages/MobileLead.vue b/frontend/src/pages/MobileLead.vue
index da7574dd..21743037 100644
--- a/frontend/src/pages/MobileLead.vue
+++ b/frontend/src/pages/MobileLead.vue
@@ -11,7 +11,14 @@
       <div class="absolute right-0">
         <Dropdown
           v-if="document.doc"
-          :options="statusOptions('lead', document, lead.data._customStatuses)"
+          :options="
+            statusOptions(
+              'lead',
+              document,
+              lead.data._customStatuses,
+              triggerOnChange,
+            )
+          "
         >
           <template #default="{ open }">
             <Button :label="document.doc.status">
@@ -461,7 +468,10 @@ async function convertToDeal() {
   }
 }
 
-const { assignees, document } = useDocument('CRM Lead', props.leadId)
+const { assignees, document, triggerOnChange } = useDocument(
+  'CRM Lead',
+  props.leadId,
+)
 
 function reloadAssignees(data) {
   if (data?.hasOwnProperty('lead_owner')) {
diff --git a/frontend/src/stores/statuses.js b/frontend/src/stores/statuses.js
index 9a63da26..0bcdbf3d 100644
--- a/frontend/src/stores/statuses.js
+++ b/frontend/src/stores/statuses.js
@@ -77,7 +77,12 @@ export const statusesStore = defineStore('crm-statuses', () => {
     return communicationStatuses[name]
   }
 
-  function statusOptions(doctype, document, statuses = []) {
+  function statusOptions(
+    doctype,
+    document,
+    statuses = [],
+    triggerOnChange = null,
+  ) {
     let statusesByName =
       doctype == 'deal' ? dealStatusesByName : leadStatusesByName
 
@@ -85,7 +90,7 @@ export const statusesStore = defineStore('crm-statuses', () => {
       statuses = document.statuses
     }
 
-    if (statuses.length) {
+    if (statuses?.length) {
       statusesByName = statuses.reduce((acc, status) => {
         acc[status] = statusesByName[status]
         return acc
@@ -98,10 +103,10 @@ export const statusesStore = defineStore('crm-statuses', () => {
         label: statusesByName[status]?.name,
         value: statusesByName[status]?.name,
         icon: () => h(IndicatorIcon, { class: statusesByName[status]?.color }),
-        onClick: () => {
+        onClick: async () => {
           capture('status_changed', { doctype, status })
           if (document) {
-            document.doc.status = statusesByName[status]?.name
+            await triggerOnChange?.('status', statusesByName[status]?.name)
             document.save.submit()
           }
         },

From 6c2ffaf8b7cc8530e76b3590e4b7f07a705b76d5 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Wed, 11 Jun 2025 19:03:22 +0530
Subject: [PATCH 06/11] feat: added enable forecasting settings

(cherry picked from commit d87a237789b55f6024f56cfd55376a2cc36b2e63)
---
 crm/fcrm/doctype/fcrm_settings/fcrm_settings.json | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/crm/fcrm/doctype/fcrm_settings/fcrm_settings.json b/crm/fcrm/doctype/fcrm_settings/fcrm_settings.json
index 250c8c29..bf2fc14b 100644
--- a/crm/fcrm/doctype/fcrm_settings/fcrm_settings.json
+++ b/crm/fcrm/doctype/fcrm_settings/fcrm_settings.json
@@ -7,6 +7,7 @@
  "field_order": [
   "defaults_tab",
   "restore_defaults",
+  "enable_forecasting",
   "branding_tab",
   "brand_name",
   "brand_logo",
@@ -28,7 +29,7 @@
   {
    "fieldname": "defaults_tab",
    "fieldtype": "Tab Break",
-   "label": "Defaults"
+   "label": "Settings"
   },
   {
    "fieldname": "branding_tab",
@@ -56,12 +57,19 @@
    "fieldname": "favicon",
    "fieldtype": "Attach",
    "label": "Favicon"
+  },
+  {
+   "default": "0",
+   "description": "It will make deal's \"Expected Closure Date\" mandatory to get accurate date forecasting insights",
+   "fieldname": "enable_forecasting",
+   "fieldtype": "Check",
+   "label": "Enable Forecasting"
   }
  ],
  "index_web_pages_for_search": 1,
  "issingle": 1,
  "links": [],
- "modified": "2025-02-20 12:38:38.088477",
+ "modified": "2025-06-11 19:02:50.134252",
  "modified_by": "Administrator",
  "module": "FCRM",
  "name": "FCRM Settings",
@@ -95,7 +103,8 @@
    "share": 1
   }
  ],
+ "row_format": "Dynamic",
  "sort_field": "creation",
  "sort_order": "DESC",
  "states": []
-}
\ No newline at end of file
+}

From d06f749544fd3e3c88ae5362b885f922cec0877a Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Wed, 11 Jun 2025 19:30:08 +0530
Subject: [PATCH 07/11] fix: set close date reqd as 1 or 0 based on
 enabled_forecasting

(cherry picked from commit 2e27c0459c183bd5c4363337d23de26f5ed72d37)
---
 .../doctype/fcrm_settings/fcrm_settings.json  |  4 ++--
 .../doctype/fcrm_settings/fcrm_settings.py    | 19 +++++++++++++++++++
 2 files changed, 21 insertions(+), 2 deletions(-)

diff --git a/crm/fcrm/doctype/fcrm_settings/fcrm_settings.json b/crm/fcrm/doctype/fcrm_settings/fcrm_settings.json
index bf2fc14b..de27e245 100644
--- a/crm/fcrm/doctype/fcrm_settings/fcrm_settings.json
+++ b/crm/fcrm/doctype/fcrm_settings/fcrm_settings.json
@@ -60,7 +60,7 @@
   },
   {
    "default": "0",
-   "description": "It will make deal's \"Expected Closure Date\" mandatory to get accurate date forecasting insights",
+   "description": "It will make deal's \"Expected Closure Date\" mandatory to get accurate forecasting insights",
    "fieldname": "enable_forecasting",
    "fieldtype": "Check",
    "label": "Enable Forecasting"
@@ -69,7 +69,7 @@
  "index_web_pages_for_search": 1,
  "issingle": 1,
  "links": [],
- "modified": "2025-06-11 19:02:50.134252",
+ "modified": "2025-06-11 19:12:16.762499",
  "modified_by": "Administrator",
  "module": "FCRM",
  "name": "FCRM Settings",
diff --git a/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py b/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py
index 53c8c77c..48da4c99 100644
--- a/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py
+++ b/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py
@@ -3,6 +3,7 @@
 
 import frappe
 from frappe import _
+from frappe.custom.doctype.property_setter.property_setter import delete_property_setter, make_property_setter
 from frappe.model.document import Document
 
 from crm.install import after_install
@@ -15,6 +16,7 @@ class FCRMSettings(Document):
 
 	def validate(self):
 		self.do_not_allow_to_delete_if_standard()
+		self.setup_forecasting()
 
 	def do_not_allow_to_delete_if_standard(self):
 		if not self.has_value_changed("dropdown_items"):
@@ -29,6 +31,23 @@ class FCRMSettings(Document):
 				return
 			frappe.throw(_("Cannot delete standard items {0}").format(", ".join(deleted_standard_items)))
 
+	def setup_forecasting(self):
+		if self.has_value_changed("enable_forecasting"):
+			if not self.enable_forecasting:
+				delete_property_setter(
+					"CRM Deal",
+					"reqd",
+					"close_date",
+				)
+			else:
+				make_property_setter(
+					"CRM Deal",
+					"close_date",
+					"reqd",
+					1 if self.enable_forecasting else 0,
+					"Check",
+				)
+
 
 def get_standard_dropdown_items():
 	return [item.get("name1") for item in frappe.get_hooks("standard_dropdown_items")]

From e9362a50eb0c4ae83bcd9fe01d03d4f384c527b4 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Fri, 13 Jun 2025 14:20:50 +0530
Subject: [PATCH 08/11] fix: set close date to now if status is Won

(cherry picked from commit a98b0e3a009f028c86ebb87385136ad0ab711a81)
---
 crm/fcrm/doctype/crm_deal/crm_deal.py | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/crm/fcrm/doctype/crm_deal/crm_deal.py b/crm/fcrm/doctype/crm_deal/crm_deal.py
index b650301e..b7e4ec83 100644
--- a/crm/fcrm/doctype/crm_deal/crm_deal.py
+++ b/crm/fcrm/doctype/crm_deal/crm_deal.py
@@ -24,6 +24,7 @@ class CRMDeal(Document):
 			self.assign_agent(self.deal_owner)
 		if self.has_value_changed("status"):
 			add_status_change_log(self)
+		self.update_close_date()
 
 	def after_insert(self):
 		if self.deal_owner:
@@ -133,6 +134,13 @@ class CRMDeal(Document):
 		if sla:
 			sla.apply(self)
 
+	def update_close_date(self):
+		"""
+		Update the close date based on the "Won" status.
+		"""
+		if self.status == "Won" and not self.close_date:
+			self.close_date = frappe.utils.nowdate()
+
 	@staticmethod
 	def default_list_data():
 		columns = [

From 232f9827869acd32a068923b1783715d2be3ca74 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Fri, 13 Jun 2025 14:42:09 +0530
Subject: [PATCH 09/11] fix: create standard forecasting script on install and
 added in patch

(cherry picked from commit b3acff8cba6fd30e15ce2d579287cbd02d9c8282)
---
 .../doctype/fcrm_settings/fcrm_settings.py    | 37 +++++++++++++++++++
 crm/install.py                                |  2 +
 crm/patches.txt                               |  2 +-
 3 files changed, 40 insertions(+), 1 deletion(-)

diff --git a/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py b/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py
index 48da4c99..0cdbc2df 100644
--- a/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py
+++ b/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py
@@ -76,3 +76,40 @@ def sync_table(key, hook):
 	crm_settings.set(key, items)
 
 	crm_settings.save()
+
+
+def create_forecasting_script():
+	if not frappe.db.exists("CRM Form Script", "Forecasting Script"):
+		script = get_forecasting_script()
+		frappe.get_doc(
+			{
+				"doctype": "CRM Form Script",
+				"name": "Forecasting Script",
+				"dt": "CRM Deal",
+				"view": "Form",
+				"script": script,
+				"enabled": 1,
+				"is_standard": 1,
+			}
+		).insert()
+
+
+def get_forecasting_script():
+	return (
+"""
+class CRMDeal {
+    async status() {
+        await this.doc.trigger('updateProbability')
+    }
+    async updateProbability() {
+        let status = await call("frappe.client.get_value", {
+            doctype: "CRM Deal Status",
+            fieldname: "probability",
+            filters: { name: this.doc.status },
+        })
+
+        this.doc.probability = status.probability
+    }
+}
+"""
+)
diff --git a/crm/install.py b/crm/install.py
index 5f25e7d0..51166471 100644
--- a/crm/install.py
+++ b/crm/install.py
@@ -5,6 +5,7 @@ import frappe
 from frappe.custom.doctype.custom_field.custom_field import create_custom_fields
 
 from crm.fcrm.doctype.crm_products.crm_products import create_product_details_script
+from crm.fcrm.doctype.fcrm_settings.fcrm_settings import create_forecasting_script
 
 
 def before_install():
@@ -361,3 +362,4 @@ def add_standard_dropdown_items():
 def add_default_scripts():
 	for doctype in ["CRM Lead", "CRM Deal"]:
 		create_product_details_script(doctype)
+	create_forecasting_script()
diff --git a/crm/patches.txt b/crm/patches.txt
index 59f37bfb..ee56be84 100644
--- a/crm/patches.txt
+++ b/crm/patches.txt
@@ -12,4 +12,4 @@ crm.patches.v1_0.create_default_sidebar_fields_layout
 crm.patches.v1_0.update_deal_quick_entry_layout
 crm.patches.v1_0.update_layouts_to_new_format
 crm.patches.v1_0.move_twilio_agent_to_telephony_agent
-crm.patches.v1_0.create_default_scripts
\ No newline at end of file
+crm.patches.v1_0.create_default_scripts # 13-06-2025
\ No newline at end of file

From 2c92612cce0e5262090070d9a027d36fdc691c31 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Fri, 13 Jun 2025 14:46:44 +0530
Subject: [PATCH 10/11] fix: circular import

(cherry picked from commit 029c16d1d0b71b83e108b479e077fbd168392870)
---
 crm/fcrm/doctype/fcrm_settings/fcrm_settings.py | 8 ++------
 crm/install.py                                  | 3 ++-
 2 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py b/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py
index 0cdbc2df..f22114f1 100644
--- a/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py
+++ b/crm/fcrm/doctype/fcrm_settings/fcrm_settings.py
@@ -95,9 +95,7 @@ def create_forecasting_script():
 
 
 def get_forecasting_script():
-	return (
-"""
-class CRMDeal {
+	return """class CRMDeal {
     async status() {
         await this.doc.trigger('updateProbability')
     }
@@ -110,6 +108,4 @@ class CRMDeal {
 
         this.doc.probability = status.probability
     }
-}
-"""
-)
+}"""
diff --git a/crm/install.py b/crm/install.py
index 51166471..5ffa5a67 100644
--- a/crm/install.py
+++ b/crm/install.py
@@ -5,7 +5,6 @@ import frappe
 from frappe.custom.doctype.custom_field.custom_field import create_custom_fields
 
 from crm.fcrm.doctype.crm_products.crm_products import create_product_details_script
-from crm.fcrm.doctype.fcrm_settings.fcrm_settings import create_forecasting_script
 
 
 def before_install():
@@ -360,6 +359,8 @@ def add_standard_dropdown_items():
 
 
 def add_default_scripts():
+	from crm.fcrm.doctype.fcrm_settings.fcrm_settings import create_forecasting_script
+
 	for doctype in ["CRM Lead", "CRM Deal"]:
 		create_product_details_script(doctype)
 	create_forecasting_script()

From 575be6595633c2f1ce33146e99ddcaadb0261664 Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Fri, 13 Jun 2025 15:02:50 +0530
Subject: [PATCH 11/11] fix: asterisk is not visible if label is big enough

(cherry picked from commit 4edfa951dcd30501d5608dac0eee20712781c9e1)
---
 frontend/src/components/SidePanelLayout.vue | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/frontend/src/components/SidePanelLayout.vue b/frontend/src/components/SidePanelLayout.vue
index b9512b46..e99d9725 100644
--- a/frontend/src/components/SidePanelLayout.vue
+++ b/frontend/src/components/SidePanelLayout.vue
@@ -44,18 +44,21 @@
                   >
                     <Tooltip :text="__(field.label)" :hoverDelay="1">
                       <div
-                        class="w-[35%] min-w-20 shrink-0 truncate text-sm text-ink-gray-5"
+                        class="w-[35%] min-w-20 shrink-0 flex items-center gap-0.5"
                       >
-                        {{ __(field.label) }}
-                        <span
+                        <div class="truncate text-sm text-ink-gray-5">
+                          {{ __(field.label) }}
+                        </div>
+                        <div
                           v-if="
                             field.reqd ||
                             (field.mandatory_depends_on &&
                               field.mandatory_via_depends_on)
                           "
                           class="text-ink-red-2"
-                          >*</span
                         >
+                          *
+                        </div>
                       </div>
                     </Tooltip>
                     <div class="flex items-center justify-between w-[65%]">