Skip to content

feat(conversions): [OCISDEV-807] add SpaceEditorWithoutVersionsWithoutTrashbin role#575

Merged
kobergj merged 2 commits intostable-8.0from
feat/OCISDEV-807-space-editor-without-versions-without-trashbin
Apr 22, 2026
Merged

feat(conversions): [OCISDEV-807] add SpaceEditorWithoutVersionsWithoutTrashbin role#575
kobergj merged 2 commits intostable-8.0from
feat/OCISDEV-807-space-editor-without-versions-without-trashbin

Conversation

@kobergj
Copy link
Copy Markdown
Collaborator

@kobergj kobergj commented Apr 22, 2026

Summary

  • Adds RoleSpaceEditorWithoutVersionsWithoutTrashbin constant and NewSpaceEditorWithoutVersionsWithoutTrashbinRole() constructor
  • Refactors NewSpaceEditorWithoutVersionsRole, NewSpaceEditorWithoutTrashbinRole and NewSpaceEditorRole to compose from the new base role
  • Adds RoleFromName case for the new role
  • Adds unit test verifying permissions are correctly set/unset

Test plan

  • go test ./pkg/conversions/... passes

🤖 Generated with Claude Code

kobergj added 2 commits April 22, 2026 09:45
…tTrashbin role

Adds the NewSpaceEditorWithoutVersionsWithoutTrashbinRole constructor and
RoleSpaceEditorWithoutVersionsWithoutTrashbin constant. This role provides
full editor permissions (create, upload, download, edit, move, delete) on
a space without access to file versions or the trashbin.
…ase role

SpaceEditorWithoutVersionsWithoutTrashbin is now the base role. The other
three SpaceEditor variants compose from it by adding the relevant permissions:
- SpaceEditorWithoutVersions adds ListRecycle + RestoreRecycleItem
- SpaceEditorWithoutTrashbin adds ListFileVersions + RestoreFileVersion
- SpaceEditor adds all four (via SpaceEditorWithoutVersions)

Signed-off-by: Julian Koberg <julian.koberg@kiteworks.com>
@kobergj kobergj merged commit a697030 into stable-8.0 Apr 22, 2026
@kobergj kobergj deleted the feat/OCISDEV-807-space-editor-without-versions-without-trashbin branch April 22, 2026 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants