From 2d1de776d4281613dd5f979dadb2b2f1ea27a7c2 Mon Sep 17 00:00:00 2001 From: GareArc Date: Tue, 6 May 2025 03:22:50 -0400 Subject: [PATCH] Revert "Merge branch 'fix/workspace-permission' into deploy/dev" This reverts commit 1d5a126baa8a522fee3c0284a231819d3a5130dc, reversing changes made to 418e0fa831225db9f8f17fe174a6fc7df923513a. --- api/services/account_service.py | 9 --------- api/services/errors/account.py | 4 ---- 2 files changed, 13 deletions(-) diff --git a/api/services/account_service.py b/api/services/account_service.py index 6a1947b137..f930ef910b 100644 --- a/api/services/account_service.py +++ b/api/services/account_service.py @@ -46,7 +46,6 @@ from services.errors.account import ( LinkAccountIntegrateError, MemberNotInTenantError, NoPermissionError, - RemoveOwnerError, RoleAlreadyAssignedError, TenantNotFoundError, ) @@ -790,9 +789,6 @@ class TenantService: ta_operator = TenantAccountJoin.query.filter_by(tenant_id=tenant.id, account_id=operator.id).first() - logging.info( - f"Tenant {tenant.id} operator {operator.id} with role {ta_operator.role} is trying to {action} member {member.id if member else None}" - ) if not ta_operator or ta_operator.role not in perms[action]: raise NoPermissionError(f"No permission to {action} member.") @@ -803,11 +799,6 @@ class TenantService: raise CannotOperateSelfError("Cannot operate self.") TenantService.check_member_permission(tenant, operator, account, "remove") - - # Additional check to ensure the target account is not owner - if TenantService.get_user_role(account, tenant) == TenantAccountRole.OWNER: - raise RemoveOwnerError("Cannot remove owner from tenant.") - ta = TenantAccountJoin.query.filter_by(tenant_id=tenant.id, account_id=account.id).first() if not ta: diff --git a/api/services/errors/account.py b/api/services/errors/account.py index cd3b24a681..5aca12ffeb 100644 --- a/api/services/errors/account.py +++ b/api/services/errors/account.py @@ -53,10 +53,6 @@ class MemberNotInTenantError(BaseServiceError): pass -class RemoveOwnerError(BaseServiceError): - pass - - class RoleAlreadyAssignedError(BaseServiceError): pass