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
This commit is contained in:
Shariq Ansari 2025-06-20 18:55:14 +05:30
parent bd7451e86f
commit ad1aee9c9e
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()