diff --git a/crm/fcrm/doctype/crm_view_settings/crm_view_settings.py b/crm/fcrm/doctype/crm_view_settings/crm_view_settings.py
index 19e0ab04..2387a9fd 100644
--- a/crm/fcrm/doctype/crm_view_settings/crm_view_settings.py
+++ b/crm/fcrm/doctype/crm_view_settings/crm_view_settings.py
@@ -65,7 +65,7 @@ def delete(name):
 
 @frappe.whitelist()
 def public(name, value):
-	if "Sales Manager" not in frappe.get_roles() or frappe.session.user != "Administrator":
+	if frappe.session.user != "Administrator" and "Sales Manager" not in frappe.get_roles():
 		frappe.throw("Not permitted", frappe.PermissionError)
 
 	doc = frappe.get_doc("CRM View Settings", name)