From b47fc5b93bb4baa9aa61a3abe5858cab0f7926db Mon Sep 17 00:00:00 2001
From: Pratik <badhepd@gmail.com>
Date: Wed, 21 May 2025 14:20:48 +0000
Subject: [PATCH 1/9] feat: handle linked docs while deleting

---
 crm/api/doc.py                                |  55 +++++++
 crm/utils/__init__.py                         | 129 +++++++++++++++
 frontend/components.d.ts                      |   1 +
 .../src/components/DeleteLinkedDocModal.vue   | 155 ++++++++++++++++++
 frontend/src/pages/Contact.vue                |  37 ++---
 frontend/src/pages/Deal.vue                   |  16 ++
 frontend/src/pages/Lead.vue                   |  17 ++
 frontend/src/pages/Organization.vue           |  38 ++---
 8 files changed, 401 insertions(+), 47 deletions(-)
 create mode 100644 frontend/src/components/DeleteLinkedDocModal.vue

diff --git a/crm/api/doc.py b/crm/api/doc.py
index 4bcbe3cc..87d22e61 100644
--- a/crm/api/doc.py
+++ b/crm/api/doc.py
@@ -10,6 +10,7 @@ from pypika import Criterion
 
 from crm.api.views import get_views
 from crm.fcrm.doctype.crm_form_script.crm_form_script import get_form_script
+from crm.utils import get_dynamic_linked_docs, get_linked_docs
 
 
 @frappe.whitelist()
@@ -726,3 +727,57 @@ def getCounts(d, doctype):
 		"FCRM Note", filters={"reference_doctype": doctype, "reference_docname": d.get("name")}
 	)
 	return d
+
+
+@frappe.whitelist()
+def getLinkedDocs(doctype, docname):
+	doc = frappe.get_doc(doctype, docname)
+	linked_docs = get_linked_docs(doc)
+	dynamic_linked_docs = get_dynamic_linked_docs(doc)
+	linked_docs.extend(dynamic_linked_docs)
+
+	return list({doc["reference_docname"]: doc for doc in linked_docs}.values())
+
+
+@frappe.whitelist()
+def removeLinkedDocReference(doctype=None, docname=None,removeAll=False,removeContact=None):
+
+	if (not doctype or not docname) and not removeAll:
+		return "Invalid doctype or docname"
+	
+	if removeAll:
+		if removeContact:
+			ref_doc = getLinkedDocs(doctype, docname)
+			for linked_doc in ref_doc:
+				removeContactLink(linked_doc["reference_doctype"], linked_doc["reference_docname"])
+
+			return "success"
+			
+		linked_docs = getLinkedDocs(doctype, docname)
+		for linked_doc in linked_docs:
+			removeDocLink(linked_doc["reference_doctype"], linked_doc["reference_docname"])
+		return "success"
+	else:
+		if removeContact:
+			removeContactLink(doctype, docname)
+			return "success"
+			
+		removeDocLink(doctype, docname)
+		return "success"
+
+def removeDocLink(doctype, docname):
+	linked_doc_data = frappe.get_doc(doctype, docname)
+	linked_doc_data.update({
+		"reference_doctype": None,
+		"reference_docname": None,
+	})
+	linked_doc_data.save(ignore_permissions=True)
+
+def removeContactLink(doctype, docname):
+	linked_doc_data = frappe.get_doc(doctype, docname)
+	linked_doc_data.update({
+		"contact": None,
+		"contacts": [],
+	})
+	linked_doc_data.save(ignore_permissions=True)
+	
\ No newline at end of file
diff --git a/crm/utils/__init__.py b/crm/utils/__init__.py
index cd4a4ff8..9b954859 100644
--- a/crm/utils/__init__.py
+++ b/crm/utils/__init__.py
@@ -1,7 +1,10 @@
+from frappe import frappe
 import phonenumbers
 from frappe.utils import floor
 from phonenumbers import NumberParseException
 from phonenumbers import PhoneNumberFormat as PNF
+from frappe.model.docstatus import DocStatus
+from frappe.model.dynamic_links import get_dynamic_link_map
 
 
 def parse_phone_number(phone_number, default_country="IN"):
@@ -93,3 +96,129 @@ def seconds_to_duration(seconds):
 		return f"{seconds}s"
 	else:
 		return "0s"
+
+# Extracted from frappe core frappe/model/delete_doc.py/check_if_doc_is_linked
+def get_linked_docs(doc, method="Delete"):
+	from frappe.model.rename_doc import get_link_fields
+
+	link_fields = get_link_fields(doc.doctype)
+	ignored_doctypes = set()
+
+	if method == "Cancel" and (doc_ignore_flags := doc.get("ignore_linked_doctypes")):
+		ignored_doctypes.update(doc_ignore_flags)
+	if method == "Delete":
+		ignored_doctypes.update(frappe.get_hooks("ignore_links_on_delete"))
+
+	docs = []
+
+	for lf in link_fields:
+		link_dt, link_field, issingle = lf["parent"], lf["fieldname"], lf["issingle"]
+		if link_dt in ignored_doctypes or (link_field == "amended_from" and method == "Cancel"):
+			continue
+
+		try:
+			meta = frappe.get_meta(link_dt)
+		except frappe.DoesNotExistError:
+			frappe.clear_last_message()
+			# This mostly happens when app do not remove their customizations, we shouldn't
+			# prevent link checks from failing in those cases
+			continue
+
+		if issingle:
+			if frappe.db.get_single_value(link_dt, link_field) == doc.name:
+				docs.append({"doc": doc.name, "link_dt": link_dt, "link_field": link_field})
+			continue
+
+		fields = ["name", "docstatus"]
+
+		if meta.istable:
+			fields.extend(["parent", "parenttype"])
+
+		for item in frappe.db.get_values(link_dt, {link_field: doc.name}, fields, as_dict=True):
+			# available only in child table cases
+			item_parent = getattr(item, "parent", None)
+			linked_parent_doctype = item.parenttype if item_parent else link_dt
+
+			if linked_parent_doctype in ignored_doctypes:
+				continue
+
+			if method != "Delete" and (method != "Cancel" or not DocStatus(item.docstatus).is_submitted()):
+				# don't raise exception if not
+				# linked to a non-cancelled doc when deleting or to a submitted doc when cancelling
+				continue
+			elif link_dt == doc.doctype and (item_parent or item.name) == doc.name:
+				# don't raise exception if not
+				# linked to same item or doc having same name as the item
+				continue
+			else:
+				reference_docname = item_parent or item.name
+				docs.append(
+					{
+						"doc": doc.name,
+						"reference_doctype": linked_parent_doctype,
+						"reference_docname": reference_docname,
+					}
+				)
+	return docs
+
+# Extracted from frappe core frappe/model/delete_doc.py/check_if_doc_is_dynamically_linked
+def get_dynamic_linked_docs(doc, method="Delete"):
+	docs = []
+	for df in get_dynamic_link_map().get(doc.doctype, []):
+		ignore_linked_doctypes = doc.get("ignore_linked_doctypes") or []
+
+		if df.parent in frappe.get_hooks("ignore_links_on_delete") or (
+			df.parent in ignore_linked_doctypes and method == "Cancel"
+		):
+			# don't check for communication and todo!
+			continue
+
+		meta = frappe.get_meta(df.parent)
+		if meta.issingle:
+			# dynamic link in single doc
+			refdoc = frappe.db.get_singles_dict(df.parent)
+			if (
+				refdoc.get(df.options) == doc.doctype
+				and refdoc.get(df.fieldname) == doc.name
+				and (
+					# linked to an non-cancelled doc when deleting
+					(method == "Delete" and not DocStatus(refdoc.docstatus).is_cancelled())
+					# linked to a submitted doc when cancelling
+					or (method == "Cancel" and DocStatus(refdoc.docstatus).is_submitted())
+				)
+			):
+				docs.append({"doc": doc.name, "reference_doctype": df.parent, "reference_docname": df.parent})
+		else:
+			# dynamic link in table
+			df["table"] = ", `parent`, `parenttype`, `idx`" if meta.istable else ""
+			for refdoc in frappe.db.sql(
+				"""select `name`, `docstatus` {table} from `tab{parent}` where
+				`{options}`=%s and `{fieldname}`=%s""".format(**df),
+				(doc.doctype, doc.name),
+				as_dict=True,
+			):
+				# linked to an non-cancelled doc when deleting
+				# or linked to a submitted doc when cancelling
+				if (method == "Delete" and not DocStatus(refdoc.docstatus).is_cancelled()) or (
+					method == "Cancel" and DocStatus(refdoc.docstatus).is_submitted()
+				):
+					reference_doctype = refdoc.parenttype if meta.istable else df.parent
+					reference_docname = refdoc.parent if meta.istable else refdoc.name
+
+					if reference_doctype in frappe.get_hooks("ignore_links_on_delete") or (
+						reference_doctype in ignore_linked_doctypes and method == "Cancel"
+					):
+						# don't check for communication and todo!
+						continue
+
+					at_position = f"at Row: {refdoc.idx}" if meta.istable else ""
+
+					docs.append(
+						{
+							"doc": doc.name,
+							"reference_doctype": reference_doctype,
+							"reference_docname": reference_docname,
+							"at_position": at_position,
+						}
+					)
+	return docs
diff --git a/frontend/components.d.ts b/frontend/components.d.ts
index 29a6ae9c..57ae1ffb 100644
--- a/frontend/components.d.ts
+++ b/frontend/components.d.ts
@@ -63,6 +63,7 @@ declare module 'vue' {
     DealsIcon: typeof import('./src/components/Icons/DealsIcon.vue')['default']
     DealsListView: typeof import('./src/components/ListViews/DealsListView.vue')['default']
     DeclinedCallIcon: typeof import('./src/components/Icons/DeclinedCallIcon.vue')['default']
+    DeleteLinkedDocModal: typeof import('./src/components/DeleteLinkedDocModal.vue')['default']
     DesendingIcon: typeof import('./src/components/Icons/DesendingIcon.vue')['default']
     DesktopLayout: typeof import('./src/components/Layouts/DesktopLayout.vue')['default']
     DetailsIcon: typeof import('./src/components/Icons/DetailsIcon.vue')['default']
diff --git a/frontend/src/components/DeleteLinkedDocModal.vue b/frontend/src/components/DeleteLinkedDocModal.vue
new file mode 100644
index 00000000..204b900b
--- /dev/null
+++ b/frontend/src/components/DeleteLinkedDocModal.vue
@@ -0,0 +1,155 @@
+<template>
+  <Dialog v-model="show" :options="{ size: 'xl' }">
+    <template #body>
+      <div class="bg-surface-modal px-4 pb-6 pt-5 sm:px-6">
+        <div class="mb-6 flex items-center justify-between">
+          <div>
+            <h3 class="text-2xl leading-6 text-ink-gray-9 font-semibold">
+              {{ __('Delete') }}
+            </h3>
+          </div>
+          <div class="flex items-center gap-1">
+            <Button variant="ghost" class="w-7" @click="show = false">
+              <FeatherIcon name="x" class="h-4 w-4" />
+            </Button>
+          </div>
+        </div>
+        <div>
+          <div v-if="linkedDocs.data?.length > 0">
+            <span>
+              {{
+                __(
+                  'Unlink these linked documents before deleting this document',
+                )
+              }}
+            </span>
+            <ul class="mt-5 space-y-1">
+              <hr />
+              <li v-for="doc in linkedDocs.data" :key="doc.name">
+                <div class="flex justify-between items-center">
+                  <span
+                    class="text-lg font-medium text-ellipsis overflow-hidden whitespace-nowrap w-full"
+                    >{{ doc.reference_doctype }} ({{
+                      doc.reference_docname
+                    }})</span
+                  >
+                  <div class="flex gap-2">
+                    <Button variant="ghost" @click="viewLinkedDoc(doc)">
+                      <div class="flex gap-1">
+                        <FeatherIcon name="external-link" class="h-4 w-4" />
+                        <span> View </span>
+                      </div>
+                    </Button>
+                    <Button variant="ghost" @click="unlinkLinkedDoc(doc)">
+                      <div class="flex gap-1">
+                        <FeatherIcon name="unlock" class="h-4 w-4" />
+                        <span> Unlink </span>
+                      </div>
+                    </Button>
+                  </div>
+                </div>
+                <hr class="my-2 w-full" />
+              </li>
+            </ul>
+          </div>
+          <div v-if="linkedDocs.data?.length == 0">
+            {{
+              __('Are you sure you want to delete {0} - {1}?', [
+                props.doctype,
+                props.docname,
+              ])
+            }}
+          </div>
+          <ErrorMessage class="mt-4" v-if="error" :message="__(error)" />
+        </div>
+      </div>
+      <div class="px-4 pb-7 pt-0 sm:px-6">
+        <div class="flex flex-row-reverse gap-2">
+          <Button
+            v-if="linkedDocs.data?.length > 0"
+            variant="solid"
+            @click="
+              unlinkLinkedDoc({
+                reference_doctype: props.doctype,
+                reference_docname: props.docname,
+                removeAll: true,
+              })
+            "
+          >
+            <div class="flex gap-1">
+              <FeatherIcon name="unlock" class="h-4 w-4" />
+              <span> Unlink all </span>
+            </div>
+          </Button>
+          <Button
+            v-if="linkedDocs.data?.length == 0"
+            variant="solid"
+            :label="__('Delete')"
+            :loading="isDealCreating"
+            @click="deleteDoc()"
+            theme="red"
+          />
+        </div>
+      </div>
+    </template>
+  </Dialog>
+</template>
+
+<script setup>
+import { createResource, call } from 'frappe-ui'
+import { useRouter } from 'vue-router'
+
+const show = defineModel()
+const router = useRouter()
+const props = defineProps({
+  name: {
+    type: String,
+    required: true,
+  },
+  doctype: {
+    type: String,
+    required: true,
+  },
+  docname: {
+    type: String,
+    required: true,
+  },
+})
+
+const linkedDocs = createResource({
+  url: 'crm.api.doc.getLinkedDocs',
+  params: {
+    doctype: props.doctype,
+    docname: props.docname,
+  },
+  auto: true,
+  validate(params) {
+    if (!params?.doctype || !params?.docname) {
+      return false
+    }
+  },
+})
+
+const viewLinkedDoc = (doc) => {
+  window.open(`/app/Form/${doc.reference_doctype}/${doc.reference_docname}`)
+}
+
+const unlinkLinkedDoc = (doc) => {
+  call('crm.api.doc.removeLinkedDocReference', {
+    doctype: doc.reference_doctype,
+    docname: doc.reference_docname,
+    removeAll: doc.removeAll,
+    removeContact: props.doctype == 'Contact',
+  }).then(() => {
+    linkedDocs.reload()
+  })
+}
+
+const deleteDoc = async () => {
+  await call('frappe.client.delete', {
+    doctype: props.doctype,
+    name: props.docname,
+  })
+  router.push({ name: props.name })
+}
+</script>
diff --git a/frontend/src/pages/Contact.vue b/frontend/src/pages/Contact.vue
index 7e0f1d4d..501a3daf 100644
--- a/frontend/src/pages/Contact.vue
+++ b/frontend/src/pages/Contact.vue
@@ -105,7 +105,7 @@
                   :label="__('Delete')"
                   theme="red"
                   size="sm"
-                  @click="deleteContact"
+                  @click="deleteContact()"
                 >
                   <template #prefix>
                     <FeatherIcon name="trash-2" class="h-4 w-4" />
@@ -173,6 +173,13 @@
     :errorMessage="errorMessage"
   />
   <AddressModal v-model="showAddressModal" v-model:address="_address" />
+  <DeleteLinkedDocModal
+    v-if="showDeleteLinkedDocModal"
+    v-model="showDeleteLinkedDocModal"
+    :doctype="'Contact'"
+    :docname="contact.data.name"
+    name="Contacts"
+  />
 </template>
 
 <script setup>
@@ -208,7 +215,6 @@ import {
 } from 'frappe-ui'
 import { ref, computed, h } from 'vue'
 import { useRoute, useRouter } from 'vue-router'
-import { errorMessage as _errorMessage } from '../utils'
 
 const { brand } = getSettings()
 const { $dialog, makeCall } = globalStore()
@@ -297,6 +303,11 @@ usePageMeta(() => {
     icon: brand.favicon,
   }
 })
+const showDeleteLinkedDocModal = ref(false)
+
+async function deleteContact() {
+  showDeleteLinkedDocModal.value = true
+}
 
 function validateFile(file) {
   let extn = file.name.split('.').pop().toLowerCase()
@@ -315,28 +326,6 @@ async function changeContactImage(file) {
   contact.reload()
 }
 
-async function deleteContact() {
-  $dialog({
-    title: __('Delete contact'),
-    message: __('Are you sure you want to delete this contact?'),
-    actions: [
-      {
-        label: __('Delete'),
-        theme: 'red',
-        variant: 'solid',
-        async onClick(close) {
-          await call('frappe.client.delete', {
-            doctype: 'Contact',
-            name: props.contactId,
-          })
-          close()
-          router.push({ name: 'Contacts' })
-        },
-      },
-    ],
-  })
-}
-
 const tabIndex = ref(0)
 const tabs = [
   {
diff --git a/frontend/src/pages/Deal.vue b/frontend/src/pages/Deal.vue
index e61c0205..37bc2c60 100644
--- a/frontend/src/pages/Deal.vue
+++ b/frontend/src/pages/Deal.vue
@@ -8,6 +8,11 @@
       </Breadcrumbs>
     </template>
     <template #right-header>
+      <Button
+        :label="__('Delete')"
+        variant="subtle"
+        @click="deleteDealWithModal()"
+      />
       <CustomActions
         v-if="deal.data._customActions?.length"
         :actions="deal.data._customActions"
@@ -299,6 +304,13 @@
       }
     "
   />
+  <DeleteLinkedDocModal
+    v-if="showDeleteLinkedDocModal"
+    v-model="showDeleteLinkedDocModal"
+    :doctype="'CRM Deal'"
+    :docname="props.dealId"
+    name="Deals"
+  />
 </template>
 <script setup>
 import ErrorPage from '@/components/ErrorPage.vue'
@@ -447,7 +459,11 @@ const reload = ref(false)
 const showOrganizationModal = ref(false)
 const showFilesUploader = ref(false)
 const _organization = ref({})
+const showDeleteLinkedDocModal = ref(false)
 
+async function deleteDealWithModal() {
+  showDeleteLinkedDocModal.value = true
+}
 function updateDeal(fieldname, value, callback) {
   value = Array.isArray(fieldname) ? '' : value
 
diff --git a/frontend/src/pages/Lead.vue b/frontend/src/pages/Lead.vue
index 8499616e..1245e5b9 100644
--- a/frontend/src/pages/Lead.vue
+++ b/frontend/src/pages/Lead.vue
@@ -34,6 +34,11 @@
           </Button>
         </template>
       </Dropdown>
+      <Button
+        :label="__('Delete')"
+        variant="subtle"
+        @click="deleteLeadWithModal(lead.data.name)"
+      />
       <Button
         :label="__('Convert to Deal')"
         variant="solid"
@@ -311,6 +316,13 @@
       }
     "
   />
+  <DeleteLinkedDocModal
+    v-if="showDeleteLinkedDocModal"
+    v-model="showDeleteLinkedDocModal"
+    :doctype="'CRM Lead'"
+    :docname="props.leadId"
+    name="Leads"
+  />
 </template>
 <script setup>
 import ErrorPage from '@/components/ErrorPage.vue'
@@ -400,6 +412,7 @@ const props = defineProps({
 
 const errorTitle = ref('')
 const errorMessage = ref('')
+const showDeleteLinkedDocModal = ref(false)
 
 const lead = createResource({
   url: 'crm.fcrm.doctype.crm_lead.api.get_lead',
@@ -606,6 +619,10 @@ async function deleteLead(name) {
   router.push({ name: 'Leads' })
 }
 
+async function deleteLeadWithModal(name) {
+  showDeleteLinkedDocModal.value = true
+}
+
 // Convert to Deal
 const showConvertToDealModal = ref(false)
 const existingContactChecked = ref(false)
diff --git a/frontend/src/pages/Organization.vue b/frontend/src/pages/Organization.vue
index 0e8a4238..2690b414 100644
--- a/frontend/src/pages/Organization.vue
+++ b/frontend/src/pages/Organization.vue
@@ -83,7 +83,7 @@
                   :label="__('Delete')"
                   theme="red"
                   size="sm"
-                  @click="deleteOrganization"
+                  @click="deleteOrganization()"
                 >
                   <template #prefix>
                     <FeatherIcon name="trash-2" class="h-4 w-4" />
@@ -170,6 +170,13 @@
     doctype="CRM Organization"
   />
   <AddressModal v-model="showAddressModal" v-model:address="_address" />
+  <DeleteLinkedDocModal
+    v-if="showDeleteLinkedDocModal"
+    v-model="showDeleteLinkedDocModal"
+    :doctype="'CRM Organization'"
+    :docname="props.organizationId"
+    name="Organizations"
+  />
 </template>
 
 <script setup>
@@ -209,6 +216,7 @@ import {
 } from 'frappe-ui'
 import { h, computed, ref } from 'vue'
 import { useRoute, useRouter } from 'vue-router'
+import DeleteLinkedDocModal from '@/components/DeleteLinkedDocModal.vue'
 
 const props = defineProps({
   organizationId: {
@@ -230,6 +238,8 @@ const router = useRouter()
 const errorTitle = ref('')
 const errorMessage = ref('')
 
+const showDeleteLinkedDocModal = ref(false)
+
 const organization = createDocumentResource({
   doctype: 'CRM Organization',
   name: props.organizationId,
@@ -294,6 +304,10 @@ usePageMeta(() => {
   }
 })
 
+async function deleteOrganization() {
+  showDeleteLinkedDocModal.value = true
+}
+
 function validateFile(file) {
   let extn = file.name.split('.').pop().toLowerCase()
   if (!['png', 'jpg', 'jpeg'].includes(extn)) {
@@ -311,28 +325,6 @@ async function changeOrganizationImage(file) {
   organization.reload()
 }
 
-async function deleteOrganization() {
-  $dialog({
-    title: __('Delete organization'),
-    message: __('Are you sure you want to delete this organization?'),
-    actions: [
-      {
-        label: __('Delete'),
-        theme: 'red',
-        variant: 'solid',
-        async onClick(close) {
-          await call('frappe.client.delete', {
-            doctype: 'CRM Organization',
-            name: props.organizationId,
-          })
-          close()
-          router.push({ name: 'Organizations' })
-        },
-      },
-    ],
-  })
-}
-
 function website(url) {
   return url && url.replace(/^(?:https?:\/\/)?(?:www\.)?/i, '')
 }

From 4a836a58ee7168ccd15d48c80b9bdfd40ddd891b Mon Sep 17 00:00:00 2001
From: Pratik <badhepd@gmail.com>
Date: Thu, 22 May 2025 07:15:22 +0000
Subject: [PATCH 2/9] feat: handle bulk delete

---
 crm/api/doc.py                              | 20 +++++++++++++++++---
 frontend/src/components/ListBulkActions.vue |  2 +-
 2 files changed, 18 insertions(+), 4 deletions(-)

diff --git a/crm/api/doc.py b/crm/api/doc.py
index 87d22e61..109831e9 100644
--- a/crm/api/doc.py
+++ b/crm/api/doc.py
@@ -741,7 +741,7 @@ def getLinkedDocs(doctype, docname):
 
 @frappe.whitelist()
 def removeLinkedDocReference(doctype=None, docname=None,removeAll=False,removeContact=None):
-
+	
 	if (not doctype or not docname) and not removeAll:
 		return "Invalid doctype or docname"
 	
@@ -750,10 +750,11 @@ def removeLinkedDocReference(doctype=None, docname=None,removeAll=False,removeCo
 			ref_doc = getLinkedDocs(doctype, docname)
 			for linked_doc in ref_doc:
 				removeContactLink(linked_doc["reference_doctype"], linked_doc["reference_docname"])
-
 			return "success"
 			
+		
 		linked_docs = getLinkedDocs(doctype, docname)
+
 		for linked_doc in linked_docs:
 			removeDocLink(linked_doc["reference_doctype"], linked_doc["reference_docname"])
 		return "success"
@@ -780,4 +781,17 @@ def removeContactLink(doctype, docname):
 		"contacts": [],
 	})
 	linked_doc_data.save(ignore_permissions=True)
-	
\ No newline at end of file
+	
+@frappe.whitelist()
+def deleteBulkDocs(doctype, items):
+	from frappe.desk.reportview import delete_bulk
+
+	items = frappe.parse_json(items)
+	for doc in items:
+		removeLinkedDocReference(doctype, doc, removeAll=True,removeContact=doctype=="Contact")
+	
+	if len(items) > 10:
+		frappe.enqueue("frappe.desk.reportview.delete_bulk", doctype=doctype, items=items)
+	else:
+		delete_bulk(doctype, items)
+	return "success"
\ No newline at end of file
diff --git a/frontend/src/components/ListBulkActions.vue b/frontend/src/components/ListBulkActions.vue
index 76b41650..227b1ec0 100644
--- a/frontend/src/components/ListBulkActions.vue
+++ b/frontend/src/components/ListBulkActions.vue
@@ -102,7 +102,7 @@ function deleteValues(selections, unselectAll) {
         theme: 'red',
         onClick: (close) => {
           capture('bulk_delete')
-          call('frappe.desk.reportview.delete_items', {
+          call('crm.api.doc.deleteBulkDocs', {
             items: JSON.stringify(Array.from(selections)),
             doctype: props.doctype,
           }).then(() => {

From af41469d5849390d6db5ac880ecbcc7dd65bf267 Mon Sep 17 00:00:00 2001
From: Pratik <badhepd@gmail.com>
Date: Mon, 26 May 2025 15:24:24 +0000
Subject: [PATCH 3/9] feat: add list view & handle bulk delete, unlink

---
 crm/api/doc.py                                |  79 ++++---
 frontend/components.d.ts                      |   6 +
 .../components/BulkDeleteLinkedDocModal.vue   | 167 ++++++++++++++
 .../src/components/DeleteLinkedDocModal.vue   | 217 ++++++++++++++----
 frontend/src/components/ListBulkActions.vue   |  59 ++---
 .../ListViews/LinkedDocsListView.vue          | 139 +++++++++++
 frontend/src/pages/CallLogs.vue               |  17 +-
 frontend/src/pages/Notes.vue                  |  17 ++
 frontend/src/pages/Tasks.vue                  |  14 +-
 9 files changed, 604 insertions(+), 111 deletions(-)
 create mode 100644 frontend/src/components/BulkDeleteLinkedDocModal.vue
 create mode 100644 frontend/src/components/ListViews/LinkedDocsListView.vue

diff --git a/crm/api/doc.py b/crm/api/doc.py
index 109831e9..f242c249 100644
--- a/crm/api/doc.py
+++ b/crm/api/doc.py
@@ -734,37 +734,28 @@ def getLinkedDocs(doctype, docname):
 	doc = frappe.get_doc(doctype, docname)
 	linked_docs = get_linked_docs(doc)
 	dynamic_linked_docs = get_dynamic_linked_docs(doc)
+	
 	linked_docs.extend(dynamic_linked_docs)
-
-	return list({doc["reference_docname"]: doc for doc in linked_docs}.values())
-
-
-@frappe.whitelist()
-def removeLinkedDocReference(doctype=None, docname=None,removeAll=False,removeContact=None):
+	linked_docs = list({doc["reference_docname"]: doc for doc in linked_docs}.values())
 	
-	if (not doctype or not docname) and not removeAll:
-		return "Invalid doctype or docname"
-	
-	if removeAll:
-		if removeContact:
-			ref_doc = getLinkedDocs(doctype, docname)
-			for linked_doc in ref_doc:
-				removeContactLink(linked_doc["reference_doctype"], linked_doc["reference_docname"])
-			return "success"
-			
+	docs_data = []
+	for doc in linked_docs:
+		data = frappe.get_doc(doc["reference_doctype"], doc["reference_docname"])
+		title = data.get("title")
+		if data.doctype == "CRM Call Log":
+			title = f"CRM Call Log - from {data.get('from')} to {data.get('to')}"
 		
-		linked_docs = getLinkedDocs(doctype, docname)
+		if data.doctype == "CRM Deal":
+			title = data.get("organization")
+		
+		docs_data.append({
+			"doc": data.doctype,
+			"title": title or data.get("name"),
+			"reference_docname": doc["reference_docname"],
+			"reference_doctype": doc["reference_doctype"],
+		})
+	return docs_data
 
-		for linked_doc in linked_docs:
-			removeDocLink(linked_doc["reference_doctype"], linked_doc["reference_docname"])
-		return "success"
-	else:
-		if removeContact:
-			removeContactLink(doctype, docname)
-			return "success"
-			
-		removeDocLink(doctype, docname)
-		return "success"
 
 def removeDocLink(doctype, docname):
 	linked_doc_data = frappe.get_doc(doctype, docname)
@@ -781,17 +772,45 @@ def removeContactLink(doctype, docname):
 		"contacts": [],
 	})
 	linked_doc_data.save(ignore_permissions=True)
+
+@frappe.whitelist()
+def removeLinkedDocReference(items, removeContact=None, delete=False):
+
+	if isinstance(items, str):
+		items = frappe.parse_json(items)
+
+	for item in items:
+		if removeContact:
+			removeContactLink(item["doctype"], item["docname"])
+		else:
+			removeDocLink(item["doctype"], item["docname"])
+		
+		if delete:
+			frappe.delete_doc(item["doctype"], item["docname"])
+
+	return "success"
+
 	
 @frappe.whitelist()
-def deleteBulkDocs(doctype, items):
+def deleteBulkDocs(doctype, items, deleteLinked=False):
 	from frappe.desk.reportview import delete_bulk
 
 	items = frappe.parse_json(items)
 	for doc in items:
-		removeLinkedDocReference(doctype, doc, removeAll=True,removeContact=doctype=="Contact")
+		linked_docs = getLinkedDocs(doctype, doc)
+		for linked_doc in linked_docs:
+			removeLinkedDocReference([
+				{
+					"doctype": linked_doc["reference_doctype"],
+					"docname": linked_doc["reference_docname"],
+				}
+			] ,
+			removeContact=doctype=="Contact",
+			delete=deleteLinked
+			)
 	
 	if len(items) > 10:
 		frappe.enqueue("frappe.desk.reportview.delete_bulk", doctype=doctype, items=items)
 	else:
 		delete_bulk(doctype, items)
-	return "success"
\ No newline at end of file
+	return "success"
diff --git a/frontend/components.d.ts b/frontend/components.d.ts
index 57ae1ffb..ab231f47 100644
--- a/frontend/components.d.ts
+++ b/frontend/components.d.ts
@@ -8,6 +8,7 @@ export {}
 /* prettier-ignore */
 declare module 'vue' {
   export interface GlobalComponents {
+    AboutModal: typeof import('./src/components/Modals/AboutModal.vue')['default']
     Activities: typeof import('./src/components/Activities/Activities.vue')['default']
     ActivityHeader: typeof import('./src/components/Activities/ActivityHeader.vue')['default']
     ActivityIcon: typeof import('./src/components/Icons/ActivityIcon.vue')['default']
@@ -29,6 +30,7 @@ declare module 'vue' {
     Autocomplete: typeof import('./src/components/frappe-ui/Autocomplete.vue')['default']
     AvatarIcon: typeof import('./src/components/Icons/AvatarIcon.vue')['default']
     BrandLogo: typeof import('./src/components/BrandLogo.vue')['default']
+    BulkDeleteLinkedDocModal: typeof import('./src/components/BulkDeleteLinkedDocModal.vue')['default']
     CalendarIcon: typeof import('./src/components/Icons/CalendarIcon.vue')['default']
     CallArea: typeof import('./src/components/Activities/CallArea.vue')['default']
     CallLogDetailModal: typeof import('./src/components/Modals/CallLogDetailModal.vue')['default']
@@ -52,6 +54,7 @@ declare module 'vue' {
     ContactsIcon: typeof import('./src/components/Icons/ContactsIcon.vue')['default']
     ContactsListView: typeof import('./src/components/ListViews/ContactsListView.vue')['default']
     ConvertIcon: typeof import('./src/components/Icons/ConvertIcon.vue')['default']
+    copy: typeof import('./src/components/DeleteLinkedDocModal copy.vue')['default']
     CountUpTimer: typeof import('./src/components/CountUpTimer.vue')['default']
     CreateDocumentModal: typeof import('./src/components/Modals/CreateDocumentModal.vue')['default']
     CRMLogo: typeof import('./src/components/Icons/CRMLogo.vue')['default']
@@ -146,12 +149,14 @@ declare module 'vue' {
     LeadsListView: typeof import('./src/components/ListViews/LeadsListView.vue')['default']
     LightningIcon: typeof import('./src/components/Icons/LightningIcon.vue')['default']
     Link: typeof import('./src/components/Controls/Link.vue')['default']
+    LinkedDocsListView: typeof import('./src/components/ListViews/LinkedDocsListView.vue')['default']
     LinkIcon: typeof import('./src/components/Icons/LinkIcon.vue')['default']
     ListBulkActions: typeof import('./src/components/ListBulkActions.vue')['default']
     ListIcon: typeof import('./src/components/Icons/ListIcon.vue')['default']
     ListRows: typeof import('./src/components/ListViews/ListRows.vue')['default']
     LoadingIndicator: typeof import('./src/components/Icons/LoadingIndicator.vue')['default']
     LucideCalendar: typeof import('~icons/lucide/calendar')['default']
+    LucideInfo: typeof import('~icons/lucide/info')['default']
     LucidePlus: typeof import('~icons/lucide/plus')['default']
     MarkAsDoneIcon: typeof import('./src/components/Icons/MarkAsDoneIcon.vue')['default']
     MaximizeIcon: typeof import('./src/components/Icons/MaximizeIcon.vue')['default']
@@ -222,6 +227,7 @@ declare module 'vue' {
     TaskPriorityIcon: typeof import('./src/components/Icons/TaskPriorityIcon.vue')['default']
     TasksListView: typeof import('./src/components/ListViews/TasksListView.vue')['default']
     TaskStatusIcon: typeof import('./src/components/Icons/TaskStatusIcon.vue')['default']
+    TelegramIcon: typeof import('./src/components/Icons/TelegramIcon.vue')['default']
     TelephonySettings: typeof import('./src/components/Settings/TelephonySettings.vue')['default']
     TerritoryIcon: typeof import('./src/components/Icons/TerritoryIcon.vue')['default']
     TwilioCallUI: typeof import('./src/components/Telephony/TwilioCallUI.vue')['default']
diff --git a/frontend/src/components/BulkDeleteLinkedDocModal.vue b/frontend/src/components/BulkDeleteLinkedDocModal.vue
new file mode 100644
index 00000000..609f4ef2
--- /dev/null
+++ b/frontend/src/components/BulkDeleteLinkedDocModal.vue
@@ -0,0 +1,167 @@
+<template>
+  <Dialog v-model="show" :options="{ size: 'xl' }">
+    <template #body>
+      <div class="bg-surface-modal px-4 pb-6 pt-5 sm:px-6">
+        <div class="mb-6 flex items-center justify-between">
+          <div>
+            <h3 class="text-2xl leading-6 text-ink-gray-9 font-semibold">
+              {{ __('Delete') }}
+            </h3>
+          </div>
+          <div class="flex items-center gap-1">
+            <Button variant="ghost" class="w-7" @click="show = false">
+              <FeatherIcon name="x" class="h-4 w-4" />
+            </Button>
+          </div>
+        </div>
+        <div>
+          <div>
+            {{
+              __('Are you sure you want to delete {0} items?', [
+                props.items?.length,
+              ])
+            }}
+          </div>
+          <ErrorMessage class="mt-4" v-if="error" :message="__(error)" />
+        </div>
+      </div>
+      <div class="px-4 pb-7 pt-0 sm:px-6">
+        <div class="flex flex-row-reverse gap-2">
+          <Button theme="red" variant="solid" @click="confirmDelete()">
+            <div class="flex gap-1">
+              <FeatherIcon name="trash" class="h-4 w-4" />
+              <span>
+                {{ __('Delete {0} items', [props.items.length]) }}
+              </span>
+            </div>
+          </Button>
+          <Button variant="solid" @click="confirmUnlink()">
+            <div class="flex gap-1">
+              <FeatherIcon name="unlock" class="h-4 w-4" />
+              <span>
+                {{ __('Unlink and delete {0} items', [props.items.length]) }}
+              </span>
+            </div>
+          </Button>
+        </div>
+      </div>
+    </template>
+    <template #body v-if="confirmDeleteInfo.show">
+      <div class="bg-surface-modal px-4 pb-6 pt-5 sm:px-6">
+        <div class="mb-6 flex items-center justify-between">
+          <div>
+            <h3 class="text-2xl leading-6 text-ink-gray-9 font-semibold">
+              {{ __('Delete') }}
+            </h3>
+          </div>
+          <div class="flex items-center gap-1">
+            <Button variant="ghost" class="w-7" @click="show = false">
+              <FeatherIcon name="x" class="h-4 w-4" />
+            </Button>
+          </div>
+        </div>
+        <div>
+          <div>
+            {{
+              confirmDeleteInfo.delete
+                ? __(
+                    'This will delete selected items and items linked to it, are you sure?',
+                  )
+                : __(
+                    'This will delete selected items and unlink linked items to it, are you sure?',
+                  )
+            }}
+          </div>
+          <ErrorMessage class="mt-4" v-if="error" :message="__(error)" />
+        </div>
+      </div>
+      <div class="px-4 pb-7 pt-0 sm:px-6">
+        <div class="flex flex-row-reverse gap-2">
+          <Button variant="solid" theme="red" @click="deleteDocs()">
+            <div class="flex gap-1">
+              <span>
+                {{
+                  confirmDeleteInfo.delete
+                    ? __('Delete')
+                    : __('Unlink and delete')
+                }}
+              </span>
+            </div>
+          </Button>
+          <Button variant="subtle" @click="confirmDeleteInfo.show = false">
+            <div class="flex gap-1">
+              <span>
+                {{ __('Cancel') }}
+              </span>
+            </div>
+          </Button>
+        </div>
+      </div>
+    </template>
+  </Dialog>
+</template>
+
+<script setup>
+import { call } from 'frappe-ui'
+import { ref } from 'vue'
+
+const show = defineModel()
+const props = defineProps({
+  doctype: {
+    type: String,
+    required: true,
+  },
+  items: {
+    type: Array,
+    required: true,
+  },
+  reload: {
+    type: Function,
+    required: true,
+  },
+})
+
+const confirmDeleteInfo = ref({
+  show: false,
+  title: '',
+  message: '',
+  delete: false,
+})
+
+const confirmDelete = () => {
+  confirmDeleteInfo.value = {
+    show: true,
+    title: __('Delete'),
+    message: __('Are you sure you want to delete {0} linked doc(s)?', [
+      props.items.length,
+    ]),
+    delete: true,
+  }
+}
+
+const confirmUnlink = () => {
+  confirmDeleteInfo.value = {
+    show: true,
+    title: __('Unlink'),
+    message: __('Are you sure you want to unlink {0} linked doc(s)?', [
+      props.items.length,
+    ]),
+    delete: false,
+  }
+}
+
+const deleteDocs = () => {
+  call('crm.api.doc.deleteBulkDocs', {
+    items: props.items,
+    doctype: props.doctype,
+    deleteLinked: confirmDeleteInfo.value.delete,
+  }).then(() => {
+    confirmDeleteInfo.value = {
+      show: false,
+      title: '',
+    }
+    show.value = false
+    props.reload()
+  })
+}
+</script>
diff --git a/frontend/src/components/DeleteLinkedDocModal.vue b/frontend/src/components/DeleteLinkedDocModal.vue
index 204b900b..f6ccedaf 100644
--- a/frontend/src/components/DeleteLinkedDocModal.vue
+++ b/frontend/src/components/DeleteLinkedDocModal.vue
@@ -1,6 +1,6 @@
 <template>
   <Dialog v-model="show" :options="{ size: 'xl' }">
-    <template #body>
+    <template #body v-if="!confirmDeleteInfo.show">
       <div class="bg-surface-modal px-4 pb-6 pt-5 sm:px-6">
         <div class="mb-6 flex items-center justify-between">
           <div>
@@ -15,44 +15,36 @@
           </div>
         </div>
         <div>
-          <div v-if="linkedDocs.data?.length > 0">
+          <div v-if="linkedDocs?.length > 0">
             <span>
               {{
                 __(
-                  'Unlink these linked documents before deleting this document',
+                  'Delete or unlink these linked documents before deleting this document',
                 )
               }}
             </span>
-            <ul class="mt-5 space-y-1">
-              <hr />
-              <li v-for="doc in linkedDocs.data" :key="doc.name">
-                <div class="flex justify-between items-center">
-                  <span
-                    class="text-lg font-medium text-ellipsis overflow-hidden whitespace-nowrap w-full"
-                    >{{ doc.reference_doctype }} ({{
-                      doc.reference_docname
-                    }})</span
-                  >
-                  <div class="flex gap-2">
-                    <Button variant="ghost" @click="viewLinkedDoc(doc)">
-                      <div class="flex gap-1">
-                        <FeatherIcon name="external-link" class="h-4 w-4" />
-                        <span> View </span>
-                      </div>
-                    </Button>
-                    <Button variant="ghost" @click="unlinkLinkedDoc(doc)">
-                      <div class="flex gap-1">
-                        <FeatherIcon name="unlock" class="h-4 w-4" />
-                        <span> Unlink </span>
-                      </div>
-                    </Button>
-                  </div>
-                </div>
-                <hr class="my-2 w-full" />
-              </li>
-            </ul>
+            <LinkedDocsListView
+              class="mt-4"
+              :rows="linkedDocs"
+              :columns="[
+                {
+                  label: 'Document',
+                  key: 'title',
+                },
+                {
+                  label: 'Doctype',
+                  key: 'reference_doctype',
+                  width: '30%',
+                },
+              ]"
+              @selectionsChanged="
+                (selections) => viewControls.updateSelections(selections)
+              "
+              :linkedDocsResource="linkedDocsResource"
+              :unlinkLinkedDoc="unlinkLinkedDoc"
+            />
           </div>
-          <div v-if="linkedDocs.data?.length == 0">
+          <div v-if="linkedDocs?.length == 0">
             {{
               __('Are you sure you want to delete {0} - {1}?', [
                 props.doctype,
@@ -66,23 +58,42 @@
       <div class="px-4 pb-7 pt-0 sm:px-6">
         <div class="flex flex-row-reverse gap-2">
           <Button
-            v-if="linkedDocs.data?.length > 0"
+            v-if="linkedDocs?.length > 0"
+            theme="red"
             variant="solid"
-            @click="
-              unlinkLinkedDoc({
-                reference_doctype: props.doctype,
-                reference_docname: props.docname,
-                removeAll: true,
-              })
-            "
+            @click="confirmDelete()"
           >
             <div class="flex gap-1">
-              <FeatherIcon name="unlock" class="h-4 w-4" />
-              <span> Unlink all </span>
+              <FeatherIcon name="trash" class="h-4 w-4" />
+              <span>
+                Delete and unlink
+                {{
+                  viewControls?.selections?.length == 0
+                    ? 'all'
+                    : `${viewControls?.selections?.length} item(s)`
+                }}
+              </span>
             </div>
           </Button>
           <Button
-            v-if="linkedDocs.data?.length == 0"
+            v-if="linkedDocs?.length > 0"
+            variant="solid"
+            @click="confirmUnlink()"
+          >
+            <div class="flex gap-1">
+              <FeatherIcon name="unlock" class="h-4 w-4" />
+              <span>
+                Unlink
+                {{
+                  viewControls?.selections?.length == 0
+                    ? 'all'
+                    : `${viewControls?.selections?.length} item(s)`
+                }}
+              </span>
+            </div>
+          </Button>
+          <Button
+            v-if="linkedDocs?.length == 0"
             variant="solid"
             :label="__('Delete')"
             :loading="isDealCreating"
@@ -92,12 +103,43 @@
         </div>
       </div>
     </template>
+    <template #body v-if="confirmDeleteInfo.show">
+      <div class="bg-surface-modal px-4 pb-6 pt-5 sm:px-6">
+        <div class="mb-6 flex items-center justify-between">
+          <div>
+            <h3 class="text-2xl leading-6 text-ink-gray-9 font-semibold">
+              {{ confirmDeleteInfo.title }}
+            </h3>
+          </div>
+          <div class="flex items-center gap-1">
+            <Button variant="ghost" class="w-7" @click="show = false">
+              <FeatherIcon name="x" class="h-4 w-4" />
+            </Button>
+          </div>
+        </div>
+        <div>
+          {{ confirmDeleteInfo.message }}
+        </div>
+        <div class="flex flex-row-reverse gap-2 mt-6">
+          <Button variant="ghost" @click="confirmDeleteInfo.show = false">
+            {{ __('Cancel') }}
+          </Button>
+          <Button
+            variant="solid"
+            :label="confirmDeleteInfo.title"
+            @click="removeDocLinks()"
+            theme="red"
+          />
+        </div>
+      </div>
+    </template>
   </Dialog>
 </template>
 
 <script setup>
 import { createResource, call } from 'frappe-ui'
 import { useRouter } from 'vue-router'
+import { computed, ref } from 'vue'
 
 const show = defineModel()
 const router = useRouter()
@@ -114,9 +156,23 @@ const props = defineProps({
     type: String,
     required: true,
   },
+  reload: {
+    type: Function,
+  },
+})
+const viewControls = ref({
+  selections: [],
+  updateSelections: (selections) => {
+    viewControls.value.selections = Array.from(selections || [])
+  },
 })
 
-const linkedDocs = createResource({
+const confirmDeleteInfo = ref({
+  show: false,
+  title: '',
+})
+
+const linkedDocsResource = createResource({
   url: 'crm.api.doc.getLinkedDocs',
   params: {
     doctype: props.doctype,
@@ -130,18 +186,76 @@ const linkedDocs = createResource({
   },
 })
 
-const viewLinkedDoc = (doc) => {
-  window.open(`/app/Form/${doc.reference_doctype}/${doc.reference_docname}`)
-}
+const linkedDocs = computed(() => {
+  return (
+    linkedDocsResource.data?.map((doc) => ({
+      id: doc.reference_docname,
+      ...doc,
+    })) || []
+  )
+})
 
 const unlinkLinkedDoc = (doc) => {
+  let selectedDocs = []
+  if (viewControls.value.selections.length > 0) {
+    Array.from(viewControls.value.selections).forEach((selection) => {
+      const docData = linkedDocs.value.find((d) => d.id == selection)
+      selectedDocs.push({
+        doctype: docData.reference_doctype,
+        docname: docData.reference_docname,
+      })
+    })
+  } else {
+    selectedDocs = linkedDocs.value.map((doc) => ({
+      doctype: doc.reference_doctype,
+      docname: doc.reference_docname,
+    }))
+  }
+
   call('crm.api.doc.removeLinkedDocReference', {
-    doctype: doc.reference_doctype,
-    docname: doc.reference_docname,
-    removeAll: doc.removeAll,
+    items: selectedDocs,
     removeContact: props.doctype == 'Contact',
+    delete: doc.delete,
   }).then(() => {
-    linkedDocs.reload()
+    linkedDocsResource.reload()
+    confirmDeleteInfo.value = {
+      show: false,
+      title: '',
+    }
+  })
+}
+
+const confirmDelete = () => {
+  const items =
+    viewControls.value.selections.length == 0
+      ? 'all'
+      : viewControls.value.selections.length
+  confirmDeleteInfo.value = {
+    show: true,
+    title: __('Delete'),
+    message: __('Are you sure you want to delete {0} linked item(s)?', [items]),
+    delete: true,
+  }
+}
+
+const confirmUnlink = () => {
+  const items =
+    viewControls.value.selections.length == 0
+      ? 'all'
+      : viewControls.value.selections.length
+  confirmDeleteInfo.value = {
+    show: true,
+    title: __('Unlink'),
+    message: __('Are you sure you want to unlink {0} linked item(s)?', [items]),
+    delete: false,
+  }
+}
+
+const removeDocLinks = () => {
+  unlinkLinkedDoc({
+    reference_doctype: props.doctype,
+    reference_docname: props.docname,
+    delete: confirmDeleteInfo.value.delete,
   })
 }
 
@@ -151,5 +265,6 @@ const deleteDoc = async () => {
     name: props.docname,
   })
   router.push({ name: props.name })
+  props?.reload?.()
 }
 </script>
diff --git a/frontend/src/components/ListBulkActions.vue b/frontend/src/components/ListBulkActions.vue
index 227b1ec0..75026362 100644
--- a/frontend/src/components/ListBulkActions.vue
+++ b/frontend/src/components/ListBulkActions.vue
@@ -14,6 +14,20 @@
     :doctype="doctype"
     @reload="reload"
   />
+  <DeleteLinkedDocModal
+    v-if="showDeleteDocModal.showLinkedDocsModal"
+    v-model="showDeleteDocModal.showLinkedDocsModal"
+    :doctype="props.doctype"
+    :docname="showDeleteDocModal.docname"
+    :reload="reload"
+  />
+  <BulkDeleteLinkedDocModal
+    v-if="showDeleteDocModal.showDeleteModal"
+    v-model="showDeleteDocModal.showDeleteModal"
+    :doctype="props.doctype"
+    :items="showDeleteDocModal.items"
+    :reload="reload"
+  />
 </template>
 
 <script setup>
@@ -50,7 +64,11 @@ const { $dialog, $socket } = globalStore()
 const showEditModal = ref(false)
 const selectedValues = ref([])
 const unselectAllAction = ref(() => {})
-
+const showDeleteDocModal = ref({
+  showLinkedDocsModal: false,
+  showDeleteModal: false,
+  docname: null,
+})
 function editValues(selections, unselectAll) {
   selectedValues.value = selections
   showEditModal.value = true
@@ -88,33 +106,18 @@ function convertToDeal(selections, unselectAll) {
 }
 
 function deleteValues(selections, unselectAll) {
-  $dialog({
-    title: __('Delete'),
-    message: __('Are you sure you want to delete {0} item(s)?', [
-      selections.size,
-    ]),
-    variant: 'solid',
-    theme: 'red',
-    actions: [
-      {
-        label: __('Delete'),
-        variant: 'solid',
-        theme: 'red',
-        onClick: (close) => {
-          capture('bulk_delete')
-          call('crm.api.doc.deleteBulkDocs', {
-            items: JSON.stringify(Array.from(selections)),
-            doctype: props.doctype,
-          }).then(() => {
-            toast.success(__('Deleted successfully'))
-            unselectAll()
-            list.value.reload()
-            close()
-          })
-        },
-      },
-    ],
-  })
+  const selectedDocs = Array.from(selections)
+  if (selectedDocs.length == 1) {
+    showDeleteDocModal.value = {
+      showLinkedDocsModal: true,
+      docname: selectedDocs[0],
+    }
+  } else {
+    showDeleteDocModal.value = {
+      showDeleteModal: true,
+      items: selectedDocs,
+    }
+  }
 }
 
 const showAssignmentModal = ref(false)
diff --git a/frontend/src/components/ListViews/LinkedDocsListView.vue b/frontend/src/components/ListViews/LinkedDocsListView.vue
new file mode 100644
index 00000000..d7e07665
--- /dev/null
+++ b/frontend/src/components/ListViews/LinkedDocsListView.vue
@@ -0,0 +1,139 @@
+<template>
+  <ListView
+    :class="$attrs.class"
+    :columns="columns"
+    :rows="rows"
+    :options="{
+      selectable: true,
+      showTooltip: true,
+      resizeColumn: true,
+    }"
+    row-key="reference_docname"
+    @update:selections="(selections) => emit('selectionsChanged', selections)"
+    ref="listViewRef"
+  >
+    <ListHeader @columnWidthUpdated="emit('columnWidthUpdated')">
+      <ListHeaderItem
+        v-for="column in columns"
+        :key="column.key"
+        :item="column"
+        @columnWidthUpdated="emit('columnWidthUpdated', column)"
+      >
+        <Button
+          v-if="column.key == 'reference_docname'"
+          variant="ghosted"
+          class="!h-4"
+          :class="isLikeFilterApplied ? 'fill-red-500' : 'fill-white'"
+          @click="() => emit('applyLikeFilter')"
+        >
+          <HeartIcon class="h-4 w-4" />
+        </Button>
+      </ListHeaderItem>
+    </ListHeader>
+    <div class="*:mx-0 *:sm:mx-0">
+      <ListRows :rows="rows" v-slot="{ idx, column, item, row }">
+        <ListRowItem
+          :item="item"
+          @click="listViewRef.toggleRow(row['reference_docname'])"
+        >
+          <template #default="{ label }">
+            <div
+              v-if="column.key === 'title'"
+              class="truncate text-base flex gap-2"
+            >
+              <span>
+                {{ label }}
+              </span>
+              <FeatherIcon
+                name="external-link"
+                class="h-4 w-4"
+                @click.stop="viewLinkedDoc(row)"
+              />
+            </div>
+          </template>
+        </ListRowItem>
+      </ListRows>
+    </div>
+  </ListView>
+</template>
+
+<script setup>
+import HeartIcon from '@/components/Icons/HeartIcon.vue'
+import ListRows from '@/components/ListViews/ListRows.vue'
+import { ListView, ListHeader, ListHeaderItem, ListRowItem } from 'frappe-ui'
+import { ref } from 'vue'
+
+const props = defineProps({
+  rows: {
+    type: Array,
+    required: true,
+  },
+  columns: {
+    type: Array,
+    required: true,
+  },
+  linkedDocsResource: {
+    type: Object,
+    required: true,
+  },
+  unlinkLinkedDoc: {
+    type: Function,
+    required: true,
+  },
+  options: {
+    type: Object,
+    default: () => ({
+      selectable: true,
+      showTooltip: true,
+      resizeColumn: false,
+      totalCount: 0,
+      rowCount: 0,
+    }),
+  },
+})
+const emit = defineEmits([
+  'loadMore',
+  'updatePageCount',
+  'columnWidthUpdated',
+  'applyFilter',
+  'applyLikeFilter',
+  'likeDoc',
+  'selectionsChanged',
+])
+
+const listViewRef = ref(null)
+
+const viewLinkedDoc = (doc) => {
+  let page = ''
+  let id = ''
+  switch (doc.reference_doctype) {
+    case 'CRM Lead':
+      page = 'leads'
+      id = doc.reference_docname
+      break
+    case 'CRM Call Log':
+      page = 'call-logs'
+      id = `view?open=${doc.reference_docname}`
+      break
+    case 'CRM Task':
+      page = 'tasks'
+      id = `view?open=${doc.reference_docname}`
+      break
+    case 'Contact':
+      page = 'contacts'
+      id = doc.reference_docname
+      break
+    case 'Organization':
+      page = 'organizations'
+      id = doc.reference_docname
+      break
+    case 'FCRM Note':
+      page = 'notes'
+      id = `view?open=${doc.reference_docname}`
+      break
+    default:
+      break
+  }
+  window.open(`/crm/${page}/${id}`)
+}
+</script>
diff --git a/frontend/src/pages/CallLogs.vue b/frontend/src/pages/CallLogs.vue
index c39c0019..5ca66240 100644
--- a/frontend/src/pages/CallLogs.vue
+++ b/frontend/src/pages/CallLogs.vue
@@ -79,7 +79,7 @@ import CallLogDetailModal from '@/components/Modals/CallLogDetailModal.vue'
 import CallLogModal from '@/components/Modals/CallLogModal.vue'
 import { getCallLogDetail } from '@/utils/callLog'
 import { createResource } from 'frappe-ui'
-import { computed, ref } from 'vue'
+import { computed, ref, onMounted } from 'vue'
 
 const callLogsListView = ref(null)
 const showCallLogModal = ref(false)
@@ -123,4 +123,19 @@ function createCallLog() {
   callLog.value = {}
   showCallLogModal.value = true
 }
+
+const openCallLogFromURL = () => {
+  const searchParams = new URLSearchParams(window.location.search)
+  const callLogName = searchParams.get('open')
+
+  if (callLogName) {
+    showCallLog(callLogName)
+    searchParams.delete('open')
+    window.history.replaceState(null, '', window.location.pathname)
+  }
+}
+
+onMounted(() => {
+  openCallLogFromURL()
+})
 </script>
diff --git a/frontend/src/pages/Notes.vue b/frontend/src/pages/Notes.vue
index 340a8c43..0c111edd 100644
--- a/frontend/src/pages/Notes.vue
+++ b/frontend/src/pages/Notes.vue
@@ -127,6 +127,7 @@ const viewControls = ref(null)
 watch(
   () => notes.value?.data?.page_length_count,
   (val, old_value) => {
+    openNoteFromURL()
     if (!val || val === old_value) return
     updatedPageCount.value = val
   },
@@ -152,4 +153,20 @@ async function deleteNote(name) {
   })
   notes.value.reload()
 }
+
+const openNoteFromURL = () => {
+  const searchParams = new URLSearchParams(window.location.search)
+  const noteName = searchParams.get('open')
+
+  if (noteName && notes.value?.data?.data) {
+    const foundNote = notes.value.data.data.find(
+      (note) => note.name === noteName,
+    )
+    if (foundNote) {
+      editNote(foundNote)
+    }
+    searchParams.delete('open')
+    window.history.replaceState(null, '', window.location.pathname)
+  }
+}
 </script>
diff --git a/frontend/src/pages/Tasks.vue b/frontend/src/pages/Tasks.vue
index f73e1804..5a2c2f1c 100644
--- a/frontend/src/pages/Tasks.vue
+++ b/frontend/src/pages/Tasks.vue
@@ -211,7 +211,7 @@ import { getMeta } from '@/stores/meta'
 import { usersStore } from '@/stores/users'
 import { formatDate, timeAgo } from '@/utils'
 import { Tooltip, Avatar, TextEditor, Dropdown, call } from 'frappe-ui'
-import { computed, ref } from 'vue'
+import { computed, ref, watch } from 'vue'
 import { useRouter } from 'vue-router'
 
 const { getFormattedPercent, getFormattedFloat, getFormattedCurrency } =
@@ -246,6 +246,7 @@ const rows = computed(() => {
     return getKanbanRows(tasks.value.data.data, tasks.value.data.fields)
   }
 
+  openTaskFromURL()
   return parseRows(tasks.value?.data.data, tasks.value?.data.columns)
 })
 
@@ -391,4 +392,15 @@ function redirect(doctype, docname) {
   }
   router.push({ name: name, params: params })
 }
+
+const openTaskFromURL = () => {
+  const searchParams = new URLSearchParams(window.location.search)
+  const taskName = searchParams.get('open')
+
+  if (taskName && rows.value?.length) {
+    showTask(parseInt(taskName))
+    searchParams.delete('open')
+    window.history.replaceState(null, '', window.location.pathname)
+  }
+}
 </script>

From 7f5f43f0c262cb0acafec1c711448202ea60f9db Mon Sep 17 00:00:00 2001
From: Pratik <badhepd@gmail.com>
Date: Tue, 27 May 2025 05:22:30 +0000
Subject: [PATCH 4/9] style: fix dark mode styles

---
 frontend/src/components/BulkDeleteLinkedDocModal.vue | 4 ++--
 frontend/src/components/DeleteLinkedDocModal.vue     | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/frontend/src/components/BulkDeleteLinkedDocModal.vue b/frontend/src/components/BulkDeleteLinkedDocModal.vue
index 609f4ef2..5365c298 100644
--- a/frontend/src/components/BulkDeleteLinkedDocModal.vue
+++ b/frontend/src/components/BulkDeleteLinkedDocModal.vue
@@ -15,7 +15,7 @@
           </div>
         </div>
         <div>
-          <div>
+          <div class="text-ink-gray-5">
             {{
               __('Are you sure you want to delete {0} items?', [
                 props.items?.length,
@@ -61,7 +61,7 @@
           </div>
         </div>
         <div>
-          <div>
+          <div class="text-ink-gray-5">
             {{
               confirmDeleteInfo.delete
                 ? __(
diff --git a/frontend/src/components/DeleteLinkedDocModal.vue b/frontend/src/components/DeleteLinkedDocModal.vue
index f6ccedaf..6341c983 100644
--- a/frontend/src/components/DeleteLinkedDocModal.vue
+++ b/frontend/src/components/DeleteLinkedDocModal.vue
@@ -16,7 +16,7 @@
         </div>
         <div>
           <div v-if="linkedDocs?.length > 0">
-            <span>
+            <span class="text-ink-gray-5">
               {{
                 __(
                   'Delete or unlink these linked documents before deleting this document',
@@ -44,7 +44,7 @@
               :unlinkLinkedDoc="unlinkLinkedDoc"
             />
           </div>
-          <div v-if="linkedDocs?.length == 0">
+          <div v-if="linkedDocs?.length == 0" class="text-ink-gray-5">
             {{
               __('Are you sure you want to delete {0} - {1}?', [
                 props.doctype,
@@ -117,7 +117,7 @@
             </Button>
           </div>
         </div>
-        <div>
+        <div class="text-ink-gray-5">
           {{ confirmDeleteInfo.message }}
         </div>
         <div class="flex flex-row-reverse gap-2 mt-6">

From 716dc056d6e4e0e323d668c598ba0b0a4d94865b Mon Sep 17 00:00:00 2001
From: Pratik <badhepd@gmail.com>
Date: Thu, 29 May 2025 05:52:36 +0000
Subject: [PATCH 5/9] refactor: move delete button

---
 frontend/src/pages/Deal.vue | 17 ++++++++++++-----
 frontend/src/pages/Lead.vue | 17 ++++++++++++-----
 2 files changed, 24 insertions(+), 10 deletions(-)

diff --git a/frontend/src/pages/Deal.vue b/frontend/src/pages/Deal.vue
index 37bc2c60..eb2c7b01 100644
--- a/frontend/src/pages/Deal.vue
+++ b/frontend/src/pages/Deal.vue
@@ -8,11 +8,6 @@
       </Breadcrumbs>
     </template>
     <template #right-header>
-      <Button
-        :label="__('Delete')"
-        variant="subtle"
-        @click="deleteDealWithModal()"
-      />
       <CustomActions
         v-if="deal.data._customActions?.length"
         :actions="deal.data._customActions"
@@ -121,6 +116,18 @@
                 </Button>
               </div>
             </Tooltip>
+            <Tooltip :text="__('Delete')">
+              <div>
+                <Button
+                  class="h-7 w-7"
+                  @click="deleteDealWithModal(deal.data.name)"
+                  variant="subtle"
+                  theme="red"
+                >
+                  <FeatherIcon name="trash" class="h-4 w-4" />
+                </Button>
+              </div>
+            </Tooltip>
           </div>
         </div>
       </div>
diff --git a/frontend/src/pages/Lead.vue b/frontend/src/pages/Lead.vue
index 1245e5b9..88a8aeb4 100644
--- a/frontend/src/pages/Lead.vue
+++ b/frontend/src/pages/Lead.vue
@@ -34,11 +34,6 @@
           </Button>
         </template>
       </Dropdown>
-      <Button
-        :label="__('Delete')"
-        variant="subtle"
-        @click="deleteLeadWithModal(lead.data.name)"
-      />
       <Button
         :label="__('Convert to Deal')"
         variant="solid"
@@ -171,6 +166,18 @@
                     </Button>
                   </div>
                 </Tooltip>
+                <Tooltip :text="__('Delete')">
+                  <div>
+                    <Button
+                      class="h-7 w-7"
+                      @click="deleteLeadWithModal(lead.data.name)"
+                      variant="subtle"
+                      theme="red"
+                    >
+                      <FeatherIcon name="trash" class="h-4 w-4" />
+                    </Button>
+                  </div>
+                </Tooltip>
               </div>
               <ErrorMessage :message="__(error)" />
             </div>

From c70dced268deb2a96dc493a95e798523088aa5ee Mon Sep 17 00:00:00 2001
From: Pratik <badhepd@gmail.com>
Date: Thu, 29 May 2025 06:02:35 +0000
Subject: [PATCH 6/9] refactor: internationalization & code clean up

---
 frontend/src/components/BulkDeleteLinkedDocModal.vue | 2 --
 frontend/src/components/DeleteLinkedDocModal.vue     | 9 ++++-----
 2 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/frontend/src/components/BulkDeleteLinkedDocModal.vue b/frontend/src/components/BulkDeleteLinkedDocModal.vue
index 5365c298..6f28cebc 100644
--- a/frontend/src/components/BulkDeleteLinkedDocModal.vue
+++ b/frontend/src/components/BulkDeleteLinkedDocModal.vue
@@ -22,7 +22,6 @@
               ])
             }}
           </div>
-          <ErrorMessage class="mt-4" v-if="error" :message="__(error)" />
         </div>
       </div>
       <div class="px-4 pb-7 pt-0 sm:px-6">
@@ -72,7 +71,6 @@
                   )
             }}
           </div>
-          <ErrorMessage class="mt-4" v-if="error" :message="__(error)" />
         </div>
       </div>
       <div class="px-4 pb-7 pt-0 sm:px-6">
diff --git a/frontend/src/components/DeleteLinkedDocModal.vue b/frontend/src/components/DeleteLinkedDocModal.vue
index 6341c983..81f6f2e3 100644
--- a/frontend/src/components/DeleteLinkedDocModal.vue
+++ b/frontend/src/components/DeleteLinkedDocModal.vue
@@ -52,7 +52,6 @@
               ])
             }}
           </div>
-          <ErrorMessage class="mt-4" v-if="error" :message="__(error)" />
         </div>
       </div>
       <div class="px-4 pb-7 pt-0 sm:px-6">
@@ -66,10 +65,10 @@
             <div class="flex gap-1">
               <FeatherIcon name="trash" class="h-4 w-4" />
               <span>
-                Delete and unlink
+                {{ __('Delete and unlink') }}
                 {{
                   viewControls?.selections?.length == 0
-                    ? 'all'
+                    ? __('all')
                     : `${viewControls?.selections?.length} item(s)`
                 }}
               </span>
@@ -83,10 +82,10 @@
             <div class="flex gap-1">
               <FeatherIcon name="unlock" class="h-4 w-4" />
               <span>
-                Unlink
+                {{ __('Unlink') }}
                 {{
                   viewControls?.selections?.length == 0
-                    ? 'all'
+                    ? __('all')
                     : `${viewControls?.selections?.length} item(s)`
                 }}
               </span>

From 34102ef6efa0ed5e942203dde6f4a925e04c9d94 Mon Sep 17 00:00:00 2001
From: Pratik <badhepd@gmail.com>
Date: Mon, 23 Jun 2025 05:15:40 +0000
Subject: [PATCH 7/9] refactor: change labels & function names

---
 crm/api/doc.py                                | 89 ++++++++++---------
 frontend/components.d.ts                      |  1 -
 .../components/BulkDeleteLinkedDocModal.vue   |  4 +-
 .../src/components/DeleteLinkedDocModal.vue   | 24 +++--
 .../ListViews/LinkedDocsListView.vue          | 37 +++++---
 5 files changed, 92 insertions(+), 63 deletions(-)

diff --git a/crm/api/doc.py b/crm/api/doc.py
index e61d484b..74227b68 100644
--- a/crm/api/doc.py
+++ b/crm/api/doc.py
@@ -678,6 +678,7 @@ def remove_assignments(doctype, name, assignees, ignore_permissions=False):
 			ignore_permissions=ignore_permissions,
 		)
 
+
 @frappe.whitelist()
 def get_assigned_users(doctype, name, default_assigned_to=None):
 	assigned_users = frappe.get_all(
@@ -749,85 +750,93 @@ def getCounts(d, doctype):
 
 
 @frappe.whitelist()
-def getLinkedDocs(doctype, docname):
+def get_linked_docs_of_document(doctype, docname):
 	doc = frappe.get_doc(doctype, docname)
 	linked_docs = get_linked_docs(doc)
 	dynamic_linked_docs = get_dynamic_linked_docs(doc)
-	
+
 	linked_docs.extend(dynamic_linked_docs)
 	linked_docs = list({doc["reference_docname"]: doc for doc in linked_docs}.values())
-	
+
 	docs_data = []
 	for doc in linked_docs:
 		data = frappe.get_doc(doc["reference_doctype"], doc["reference_docname"])
 		title = data.get("title")
 		if data.doctype == "CRM Call Log":
-			title = f"CRM Call Log - from {data.get('from')} to {data.get('to')}"
-		
+			title = f"Call from {data.get('from')} to {data.get('to')}"
+
 		if data.doctype == "CRM Deal":
 			title = data.get("organization")
-		
-		docs_data.append({
-			"doc": data.doctype,
-			"title": title or data.get("name"),
-			"reference_docname": doc["reference_docname"],
-			"reference_doctype": doc["reference_doctype"],
-		})
+
+		docs_data.append(
+			{
+				"doc": data.doctype,
+				"title": title or data.get("name"),
+				"reference_docname": doc["reference_docname"],
+				"reference_doctype": doc["reference_doctype"],
+			}
+		)
 	return docs_data
 
 
-def removeDocLink(doctype, docname):
+def remove_doc_link(doctype, docname):
 	linked_doc_data = frappe.get_doc(doctype, docname)
-	linked_doc_data.update({
-		"reference_doctype": None,
-		"reference_docname": None,
-	})
+	linked_doc_data.update(
+		{
+			"reference_doctype": None,
+			"reference_docname": None,
+		}
+	)
 	linked_doc_data.save(ignore_permissions=True)
 
-def removeContactLink(doctype, docname):
+
+def remove_contact_link(doctype, docname):
 	linked_doc_data = frappe.get_doc(doctype, docname)
-	linked_doc_data.update({
-		"contact": None,
-		"contacts": [],
-	})
+	linked_doc_data.update(
+		{
+			"contact": None,
+			"contacts": [],
+		}
+	)
 	linked_doc_data.save(ignore_permissions=True)
 
+
 @frappe.whitelist()
-def removeLinkedDocReference(items, removeContact=None, delete=False):
-
+def remove_linked_doc_reference(items, remove_contact=None, delete=False):
 	if isinstance(items, str):
 		items = frappe.parse_json(items)
 
 	for item in items:
-		if removeContact:
-			removeContactLink(item["doctype"], item["docname"])
+		if remove_contact:
+			remove_contact_link(item["doctype"], item["docname"])
 		else:
-			removeDocLink(item["doctype"], item["docname"])
-		
+			remove_doc_link(item["doctype"], item["docname"])
+
 		if delete:
 			frappe.delete_doc(item["doctype"], item["docname"])
 
 	return "success"
 
-	
+
 @frappe.whitelist()
-def deleteBulkDocs(doctype, items, deleteLinked=False):
+def delete_bulk_docs(doctype, items, delete_linked=False):
 	from frappe.desk.reportview import delete_bulk
 
 	items = frappe.parse_json(items)
 	for doc in items:
-		linked_docs = getLinkedDocs(doctype, doc)
+		linked_docs = get_linked_docs_of_document(doctype, doc)
 		for linked_doc in linked_docs:
-			removeLinkedDocReference([
-				{
-					"doctype": linked_doc["reference_doctype"],
-					"docname": linked_doc["reference_docname"],
-				}
-			] ,
-			removeContact=doctype=="Contact",
-			delete=deleteLinked
+			remove_linked_doc_reference(
+				[
+					{
+						"doctype": linked_doc["reference_doctype"],
+						"docname": linked_doc["reference_docname"],
+					}
+				],
+				remove_contact=doctype == "Contact",
+				delete=delete_linked,
 			)
-	
+
 	if len(items) > 10:
 		frappe.enqueue("frappe.desk.reportview.delete_bulk", doctype=doctype, items=items)
 	else:
diff --git a/frontend/components.d.ts b/frontend/components.d.ts
index 09597c6d..03c0c61c 100644
--- a/frontend/components.d.ts
+++ b/frontend/components.d.ts
@@ -54,7 +54,6 @@ declare module 'vue' {
     ContactsIcon: typeof import('./src/components/Icons/ContactsIcon.vue')['default']
     ContactsListView: typeof import('./src/components/ListViews/ContactsListView.vue')['default']
     ConvertIcon: typeof import('./src/components/Icons/ConvertIcon.vue')['default']
-    copy: typeof import('./src/components/DeleteLinkedDocModal copy.vue')['default']
     CountUpTimer: typeof import('./src/components/CountUpTimer.vue')['default']
     CreateDocumentModal: typeof import('./src/components/Modals/CreateDocumentModal.vue')['default']
     CRMLogo: typeof import('./src/components/Icons/CRMLogo.vue')['default']
diff --git a/frontend/src/components/BulkDeleteLinkedDocModal.vue b/frontend/src/components/BulkDeleteLinkedDocModal.vue
index 6f28cebc..40b6cbac 100644
--- a/frontend/src/components/BulkDeleteLinkedDocModal.vue
+++ b/frontend/src/components/BulkDeleteLinkedDocModal.vue
@@ -149,10 +149,10 @@ const confirmUnlink = () => {
 }
 
 const deleteDocs = () => {
-  call('crm.api.doc.deleteBulkDocs', {
+  call('crm.api.doc.delete_bulk_docs', {
     items: props.items,
     doctype: props.doctype,
-    deleteLinked: confirmDeleteInfo.value.delete,
+    delete_linked: confirmDeleteInfo.value.delete,
   }).then(() => {
     confirmDeleteInfo.value = {
       show: false,
diff --git a/frontend/src/components/DeleteLinkedDocModal.vue b/frontend/src/components/DeleteLinkedDocModal.vue
index 81f6f2e3..224ee9b9 100644
--- a/frontend/src/components/DeleteLinkedDocModal.vue
+++ b/frontend/src/components/DeleteLinkedDocModal.vue
@@ -5,7 +5,11 @@
         <div class="mb-6 flex items-center justify-between">
           <div>
             <h3 class="text-2xl leading-6 text-ink-gray-9 font-semibold">
-              {{ __('Delete') }}
+              {{
+                linkedDocs?.length == 0
+                  ? __('Delete')
+                  : __('Delete or unlink linked documents')
+              }}
             </h3>
           </div>
           <div class="flex items-center gap-1">
@@ -32,7 +36,7 @@
                   key: 'title',
                 },
                 {
-                  label: 'Doctype',
+                  label: 'Master',
                   key: 'reference_doctype',
                   width: '30%',
                 },
@@ -65,7 +69,7 @@
             <div class="flex gap-1">
               <FeatherIcon name="trash" class="h-4 w-4" />
               <span>
-                {{ __('Delete and unlink') }}
+                {{ __('Delete') }}
                 {{
                   viewControls?.selections?.length == 0
                     ? __('all')
@@ -76,7 +80,8 @@
           </Button>
           <Button
             v-if="linkedDocs?.length > 0"
-            variant="solid"
+            variant="subtle"
+            theme="gray"
             @click="confirmUnlink()"
           >
             <div class="flex gap-1">
@@ -172,7 +177,7 @@ const confirmDeleteInfo = ref({
 })
 
 const linkedDocsResource = createResource({
-  url: 'crm.api.doc.getLinkedDocs',
+  url: 'crm.api.doc.get_linked_docs_of_document',
   params: {
     doctype: props.doctype,
     docname: props.docname,
@@ -211,9 +216,9 @@ const unlinkLinkedDoc = (doc) => {
     }))
   }
 
-  call('crm.api.doc.removeLinkedDocReference', {
+  call('crm.api.doc.remove_linked_doc_reference', {
     items: selectedDocs,
-    removeContact: props.doctype == 'Contact',
+    remove_contact: props.doctype == 'Contact',
     delete: doc.delete,
   }).then(() => {
     linkedDocsResource.reload()
@@ -231,7 +236,7 @@ const confirmDelete = () => {
       : viewControls.value.selections.length
   confirmDeleteInfo.value = {
     show: true,
-    title: __('Delete'),
+    title: __('Delete linked item'),
     message: __('Are you sure you want to delete {0} linked item(s)?', [items]),
     delete: true,
   }
@@ -244,7 +249,7 @@ const confirmUnlink = () => {
       : viewControls.value.selections.length
   confirmDeleteInfo.value = {
     show: true,
-    title: __('Unlink'),
+    title: __('Unlink linked item'),
     message: __('Are you sure you want to unlink {0} linked item(s)?', [items]),
     delete: false,
   }
@@ -256,6 +261,7 @@ const removeDocLinks = () => {
     reference_docname: props.docname,
     delete: confirmDeleteInfo.value.delete,
   })
+  viewControls.value.updateSelections([])
 }
 
 const deleteDoc = async () => {
diff --git a/frontend/src/components/ListViews/LinkedDocsListView.vue b/frontend/src/components/ListViews/LinkedDocsListView.vue
index d7e07665..d9cdd4fb 100644
--- a/frontend/src/components/ListViews/LinkedDocsListView.vue
+++ b/frontend/src/components/ListViews/LinkedDocsListView.vue
@@ -19,15 +19,6 @@
         :item="column"
         @columnWidthUpdated="emit('columnWidthUpdated', column)"
       >
-        <Button
-          v-if="column.key == 'reference_docname'"
-          variant="ghosted"
-          class="!h-4"
-          :class="isLikeFilterApplied ? 'fill-red-500' : 'fill-white'"
-          @click="() => emit('applyLikeFilter')"
-        >
-          <HeartIcon class="h-4 w-4" />
-        </Button>
       </ListHeaderItem>
     </ListHeader>
     <div class="*:mx-0 *:sm:mx-0">
@@ -50,6 +41,12 @@
                 @click.stop="viewLinkedDoc(row)"
               />
             </div>
+            <span
+              v-if="column.key === 'reference_doctype'"
+              class="truncate text-base flex gap-2"
+            >
+              {{ getDoctypeName(row.reference_doctype) }}
+            </span>
           </template>
         </ListRowItem>
       </ListRows>
@@ -58,7 +55,6 @@
 </template>
 
 <script setup>
-import HeartIcon from '@/components/Icons/HeartIcon.vue'
 import ListRows from '@/components/ListViews/ListRows.vue'
 import { ListView, ListHeader, ListHeaderItem, ListRowItem } from 'frappe-ui'
 import { ref } from 'vue'
@@ -123,7 +119,7 @@ const viewLinkedDoc = (doc) => {
       page = 'contacts'
       id = doc.reference_docname
       break
-    case 'Organization':
+    case 'CRM Organization':
       page = 'organizations'
       id = doc.reference_docname
       break
@@ -136,4 +132,23 @@ const viewLinkedDoc = (doc) => {
   }
   window.open(`/crm/${page}/${id}`)
 }
+
+const getDoctypeName = (doctype) => {
+  switch (doctype) {
+    case 'CRM Lead':
+      return 'Lead'
+    case 'CRM Call Log':
+      return 'Call Log'
+    case 'CRM Task':
+      return 'Task'
+    case 'Contact':
+      return 'Contact'
+    case 'CRM Organization':
+      return 'Organization'
+    case 'FCRM Note':
+      return 'Note'
+    default:
+      return doctype
+  }
+}
 </script>

From 480cc07cd9fc4682e7d75f8d89fe3262e998cac3 Mon Sep 17 00:00:00 2001
From: Pratik <badhepd@gmail.com>
Date: Mon, 23 Jun 2025 07:35:15 +0000
Subject: [PATCH 8/9] refactor: remove unnecessary functions & components

---
 .../src/components/DeleteLinkedDocModal.vue     | 15 ++++++++++-----
 .../components/ListViews/LinkedDocsListView.vue | 17 +----------------
 frontend/src/pages/Contact.vue                  |  1 -
 frontend/src/pages/Organization.vue             | 13 -------------
 4 files changed, 11 insertions(+), 35 deletions(-)

diff --git a/frontend/src/components/DeleteLinkedDocModal.vue b/frontend/src/components/DeleteLinkedDocModal.vue
index 224ee9b9..eb5e4694 100644
--- a/frontend/src/components/DeleteLinkedDocModal.vue
+++ b/frontend/src/components/DeleteLinkedDocModal.vue
@@ -2,7 +2,7 @@
   <Dialog v-model="show" :options="{ size: 'xl' }">
     <template #body v-if="!confirmDeleteInfo.show">
       <div class="bg-surface-modal px-4 pb-6 pt-5 sm:px-6">
-        <div class="mb-6 flex items-center justify-between">
+        <div class="mb-4 flex items-center justify-between">
           <div>
             <h3 class="text-2xl leading-6 text-ink-gray-9 font-semibold">
               {{
@@ -20,7 +20,7 @@
         </div>
         <div>
           <div v-if="linkedDocs?.length > 0">
-            <span class="text-ink-gray-5">
+            <span class="text-ink-gray-5 text-base">
               {{
                 __(
                   'Delete or unlink these linked documents before deleting this document',
@@ -48,7 +48,7 @@
               :unlinkLinkedDoc="unlinkLinkedDoc"
             />
           </div>
-          <div v-if="linkedDocs?.length == 0" class="text-ink-gray-5">
+          <div v-if="linkedDocs?.length == 0" class="text-ink-gray-5 text-base">
             {{
               __('Are you sure you want to delete {0} - {1}?', [
                 props.doctype,
@@ -121,11 +121,11 @@
             </Button>
           </div>
         </div>
-        <div class="text-ink-gray-5">
+        <div class="text-ink-gray-5 text-base">
           {{ confirmDeleteInfo.message }}
         </div>
         <div class="flex flex-row-reverse gap-2 mt-6">
-          <Button variant="ghost" @click="confirmDeleteInfo.show = false">
+          <Button variant="ghost" @click="cancel()">
             {{ __('Cancel') }}
           </Button>
           <Button
@@ -199,6 +199,11 @@ const linkedDocs = computed(() => {
   )
 })
 
+const cancel = () => {
+  confirmDeleteInfo.value.show = false
+  viewControls.value.updateSelections([])
+}
+
 const unlinkLinkedDoc = (doc) => {
   let selectedDocs = []
   if (viewControls.value.selections.length > 0) {
diff --git a/frontend/src/components/ListViews/LinkedDocsListView.vue b/frontend/src/components/ListViews/LinkedDocsListView.vue
index d9cdd4fb..9345846e 100644
--- a/frontend/src/components/ListViews/LinkedDocsListView.vue
+++ b/frontend/src/components/ListViews/LinkedDocsListView.vue
@@ -134,21 +134,6 @@ const viewLinkedDoc = (doc) => {
 }
 
 const getDoctypeName = (doctype) => {
-  switch (doctype) {
-    case 'CRM Lead':
-      return 'Lead'
-    case 'CRM Call Log':
-      return 'Call Log'
-    case 'CRM Task':
-      return 'Task'
-    case 'Contact':
-      return 'Contact'
-    case 'CRM Organization':
-      return 'Organization'
-    case 'FCRM Note':
-      return 'Note'
-    default:
-      return doctype
-  }
+  return doctype.replace(/^(CRM|FCRM)\s*/, '')
 }
 </script>
diff --git a/frontend/src/pages/Contact.vue b/frontend/src/pages/Contact.vue
index 27bc88e4..b77f5ce7 100644
--- a/frontend/src/pages/Contact.vue
+++ b/frontend/src/pages/Contact.vue
@@ -172,7 +172,6 @@
     :errorTitle="errorTitle"
     :errorMessage="errorMessage"
   />
-  <AddressModal v-model="showAddressModal" v-model:address="_address" />
   <DeleteLinkedDocModal
     v-if="showDeleteLinkedDocModal"
     v-model="showDeleteLinkedDocModal"
diff --git a/frontend/src/pages/Organization.vue b/frontend/src/pages/Organization.vue
index f39fa9f2..769d345f 100644
--- a/frontend/src/pages/Organization.vue
+++ b/frontend/src/pages/Organization.vue
@@ -164,12 +164,6 @@
     :errorTitle="errorTitle"
     :errorMessage="errorMessage"
   />
-  <QuickEntryModal
-    v-if="showQuickEntryModal"
-    v-model="showQuickEntryModal"
-    doctype="CRM Organization"
-  />
-  <AddressModal v-model="showAddressModal" v-model:address="_address" />
   <DeleteLinkedDocModal
     v-if="showDeleteLinkedDocModal"
     v-model="showDeleteLinkedDocModal"
@@ -306,13 +300,6 @@ async function deleteOrganization() {
   showDeleteLinkedDocModal.value = true
 }
 
-function validateFile(file) {
-  let extn = file.name.split('.').pop().toLowerCase()
-  if (!['png', 'jpg', 'jpeg'].includes(extn)) {
-    return __('Only PNG and JPG images are allowed')
-  }
-}
-
 async function changeOrganizationImage(file) {
   await call('frappe.client.set_value', {
     doctype: 'CRM Organization',

From 594295b7c8668c680c74328ea82a41a974bd9b95 Mon Sep 17 00:00:00 2001
From: Pratik <badhepd@gmail.com>
Date: Mon, 23 Jun 2025 08:13:06 +0000
Subject: [PATCH 9/9] style: switch button position

---
 frontend/src/components/DeleteLinkedDocModal.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/components/DeleteLinkedDocModal.vue b/frontend/src/components/DeleteLinkedDocModal.vue
index eb5e4694..d85548f0 100644
--- a/frontend/src/components/DeleteLinkedDocModal.vue
+++ b/frontend/src/components/DeleteLinkedDocModal.vue
@@ -124,7 +124,7 @@
         <div class="text-ink-gray-5 text-base">
           {{ confirmDeleteInfo.message }}
         </div>
-        <div class="flex flex-row-reverse gap-2 mt-6">
+        <div class="flex justify-end gap-2 mt-6">
           <Button variant="ghost" @click="cancel()">
             {{ __('Cancel') }}
           </Button>