fix: added app permission check for apps page
This commit is contained in:
parent
0edde767fd
commit
703c3c504f
@ -2,7 +2,6 @@ from bs4 import BeautifulSoup
|
|||||||
import frappe
|
import frappe
|
||||||
from frappe.translate import get_all_translations
|
from frappe.translate import get_all_translations
|
||||||
from frappe.utils import cstr
|
from frappe.utils import cstr
|
||||||
from frappe.utils.telemetry import POSTHOG_HOST_FIELD, POSTHOG_PROJECT_FIELD
|
|
||||||
|
|
||||||
|
|
||||||
@frappe.whitelist(allow_guest=True)
|
@frappe.whitelist(allow_guest=True)
|
||||||
@ -47,11 +46,13 @@ def get_user_signature():
|
|||||||
content = f'<br><p class="signature">{signature}</p>'
|
content = f'<br><p class="signature">{signature}</p>'
|
||||||
return content
|
return content
|
||||||
|
|
||||||
@frappe.whitelist()
|
|
||||||
def get_posthog_settings():
|
def check_app_permission():
|
||||||
return {
|
if frappe.session.user == "Administrator":
|
||||||
"posthog_project_id": frappe.conf.get(POSTHOG_PROJECT_FIELD),
|
return True
|
||||||
"posthog_host": frappe.conf.get(POSTHOG_HOST_FIELD),
|
|
||||||
"enable_telemetry": frappe.get_system_settings("enable_telemetry"),
|
roles = frappe.get_roles()
|
||||||
"telemetry_site_age": frappe.utils.telemetry.site_age(),
|
if any(role in ["System Manager", "Sales User", "Sales Manager", "Sales Master Manager"] for role in roles):
|
||||||
}
|
return True
|
||||||
|
|
||||||
|
return False
|
||||||
@ -18,7 +18,7 @@ add_to_apps_screen = [
|
|||||||
"logo": "/assets/crm/manifest/apple-icon-180.png",
|
"logo": "/assets/crm/manifest/apple-icon-180.png",
|
||||||
"title": "CRM",
|
"title": "CRM",
|
||||||
"route": "/crm",
|
"route": "/crm",
|
||||||
# "has_permission": "crm.api.permission.has_app_permission"
|
"has_permission": "crm.api.check_app_permission",
|
||||||
}
|
}
|
||||||
]
|
]
|
||||||
|
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user