diff --git a/frappe/desk/form/document_follow.py b/frappe/desk/form/document_follow.py index 58c0e8d13f..8839d0a393 100644 --- a/frappe/desk/form/document_follow.py +++ b/frappe/desk/form/document_follow.py @@ -12,7 +12,7 @@ from frappe.utils import get_url_to_form @frappe.whitelist() def update_follow(doctype: str, doc_name: str, following: bool): if following: - return follow_document(doctype, doc_name, frappe.session.user) + return follow_document(doctype, doc_name, frappe.session.user) and True or False else: return unfollow_document(doctype, doc_name, frappe.session.user) @@ -61,7 +61,7 @@ def follow_document(doctype, doc_name, user): doc.update({"ref_doctype": doctype, "ref_docname": doc_name, "user": user}) doc.save() frappe.toast(_("Following document {}".format(doc_name))) - return True + return doc return False diff --git a/frappe/public/js/frappe/views/workspace/workspace.js b/frappe/public/js/frappe/views/workspace/workspace.js index 0311bc976c..ebec30fe09 100644 --- a/frappe/public/js/frappe/views/workspace/workspace.js +++ b/frappe/public/js/frappe/views/workspace/workspace.js @@ -54,7 +54,11 @@ frappe.views.Workspace = class Workspace { prepare_container() { let list_sidebar = $(` `).appendTo(this.wrapper.find(".layout-side-section")); this.sidebar = list_sidebar.find(".desk-sidebar"); @@ -95,6 +99,10 @@ frappe.views.Workspace = class Workspace { this.make_sidebar(); reload && this.show(); } + + this.wrapper.find(".layout-side-section .edit-sidebar-link").on("click", () => { + frappe.quick_edit("Workspace Settings"); + }); } prepare_new_and_edit() {