test/crm
1
0
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: add user is actually add role

Browse Source 
(cherry picked from commit ad1aee9c9e03f37e1deabc5b3941032e165f60a8)
This commit is contained in:
Shariq Ansari 2025-06-20 18:55:14 +05:30 committed by Mergify
parent bae37529e5
commit df605f1a9a
1 changed files with 1 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
crm/api/user.py
View File

@ -50,22 +50,7 @@ def add_user(user, role):
:param user: The name of the user to be added
:param role: The role to be assigned (Sales User or Sales Manager)
"""
frappe.only_for(["System Manager", "Sales Manager"])
if role not in ["System Manager", "Sales Manager", "Sales User"]:
frappe.throw("Cannot assign this role")
user_doc = frappe.get_doc("User", user)
if role == "System Manager":
user_doc.append_roles("System Manager", "Sales Manager", "Sales User")
elif role == "Sales Manager":
user_doc.append_roles("Sales Manager", "Sales User")
elif role == "Sales User":
user_doc.append_roles("Sales User")
update_module_in_user(user_doc, "FCRM")
user_doc.save(ignore_permissions=True)
update_user_role(user, role)
@frappe.whitelist()