Skip to content

Commit 7d7f085

Browse files
committed
refactor: update role constants to include ADMIN in permission checks
1 parent a0b71de commit 7d7f085

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

apps/folders/views/folder.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,8 @@ def post(self, request: Request, workspace_id: str, source: str):
6565
@has_permissions(
6666
lambda r, kwargs: Permission(group=Group(kwargs.get('source')), operate=Operate.READ,
6767
resource_path=f"/WORKSPACE/{kwargs.get('workspace_id')}"),
68-
RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()
68+
RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role(),
69+
RoleConstants.ADMIN
6970
)
7071
def get(self, request: Request, workspace_id: str, source: str):
7172
return result.success(FolderTreeSerializer(
@@ -111,7 +112,8 @@ def put(self, request: Request, workspace_id: str, source: str, folder_id: str):
111112
@has_permissions(
112113
lambda r, kwargs: Permission(group=Group(kwargs.get('source')), operate=Operate.READ,
113114
resource_path=f"/WORKSPACE/{kwargs.get('workspace_id')}"),
114-
RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role()
115+
RoleConstants.WORKSPACE_MANAGE.get_workspace_role(), RoleConstants.USER.get_workspace_role(),
116+
RoleConstants.ADMIN
115117
)
116118
def get(self, request: Request, workspace_id: str, source: str, folder_id: str):
117119
return result.success(FolderSerializer.Operate(

0 commit comments

Comments
 (0)