Merge pull request #42994 from frappe/mergify/bp/version-15-hotfix/pr-42993

fix: added app permission check for apps page (backport #42993)
This commit is contained in:
Shariq Ansari
2024-08-30 18:05:55 +05:30
committed by GitHub
2 changed files with 12 additions and 1 deletions

View File

@@ -2,6 +2,7 @@ import functools
import inspect
import frappe
from frappe.utils.user import is_website_user
__version__ = "15.28.2"
@@ -149,3 +150,13 @@ def allow_regional(fn):
return frappe.get_attr(overrides[function_path][-1])(*args, **kwargs)
return caller
def check_app_permission():
if frappe.session.user == "Administrator":
return True
if is_website_user():
return False
return True

View File

@@ -16,7 +16,7 @@ add_to_apps_screen = [
"logo": "/assets/erpnext/images/erpnext-logo-blue.png",
"title": "ERPNext",
"route": "/app/home",
# "has_permission": "erpnext.api.permission.has_app_permission"
"has_permission": "erpnext.check_app_permission",
}
]