From e5d5d0b6bb6c74007da312fb647d9a16b3f2e61d Mon Sep 17 00:00:00 2001
From: Shivam Mishra <scm.mymail@gmail.com>
Date: Thu, 14 May 2020 16:07:57 +0530
Subject: [PATCH] feat: allow linking dashboard in desk page

---
 frappe/desk/desktop.py                               | 2 ++
 frappe/desk/doctype/desk_shortcut/desk_shortcut.json | 4 ++--
 frappe/public/js/frappe/widgets/utils.js             | 2 ++
 frappe/public/js/frappe/widgets/widget_dialog.js     | 2 +-
 4 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/frappe/desk/desktop.py b/frappe/desk/desktop.py
index ddcfb670d4..e2fe020eca 100644
--- a/frappe/desk/desktop.py
+++ b/frappe/desk/desktop.py
@@ -127,6 +127,8 @@ class Workspace:
 			return name in self.allowed_reports
 		if item_type == "help":
 			return True
+		if item_type == "dashboard":
+			return True
 
 		return False
 
diff --git a/frappe/desk/doctype/desk_shortcut/desk_shortcut.json b/frappe/desk/doctype/desk_shortcut/desk_shortcut.json
index 550ea609c8..f3fd546a77 100644
--- a/frappe/desk/doctype/desk_shortcut/desk_shortcut.json
+++ b/frappe/desk/doctype/desk_shortcut/desk_shortcut.json
@@ -23,7 +23,7 @@
    "fieldtype": "Select",
    "in_list_view": 1,
    "label": "Type",
-   "options": "DocType\nReport\nPage",
+   "options": "DocType\nReport\nPage\nDashboard",
    "reqd": 1
   },
   {
@@ -88,7 +88,7 @@
  ],
  "istable": 1,
  "links": [],
- "modified": "2020-05-13 19:26:34.229669",
+ "modified": "2020-05-14 16:02:15.420993",
  "modified_by": "Administrator",
  "module": "Desk",
  "name": "Desk Shortcut",
diff --git a/frappe/public/js/frappe/widgets/utils.js b/frappe/public/js/frappe/widgets/utils.js
index 0d93bb3784..f336335d1c 100644
--- a/frappe/public/js/frappe/widgets/utils.js
+++ b/frappe/public/js/frappe/widgets/utils.js
@@ -24,6 +24,8 @@ function generate_route(item) {
 			route = "List/" + item.doctype + "/Report/" + item.name;
 		} else if (type === "page") {
 			route = item.name;
+		} else if (type === "dashboard") {
+			route = "dashboard/" + item.name;
 		}
 
 		route = "#" + route;
diff --git a/frappe/public/js/frappe/widgets/widget_dialog.js b/frappe/public/js/frappe/widgets/widget_dialog.js
index 31215a40c3..5c44533b37 100644
--- a/frappe/public/js/frappe/widgets/widget_dialog.js
+++ b/frappe/public/js/frappe/widgets/widget_dialog.js
@@ -145,7 +145,7 @@ class ShortcutDialog extends WidgetDialog {
 				fieldname: "type",
 				label: "Type",
 				reqd: 1,
-				options: "DocType\nReport\nPage",
+				options: "DocType\nReport\nPage\nDashboard",
 				onchange: () => {
 					if (this.dialog.get_value("type") == "DocType") {
 						this.dialog.fields_dict.link_to.get_query = () => {