Skip to content

feat: add more roles - #671

Merged
micbar merged 1 commit into
mainfrom
add-roles
Jun 11, 2026
Merged

micbar merged 1 commit into
mainfrom
add-roles

Conversation

@micbar

@micbar micbar commented Jun 10, 2026 •

Copy link
Copy Markdown
Member

This adds 3 more Roles:

  1. ViewerWithVersions
  2. FileEditorWithVersions
  3. EditorWithVersions

@micbar
micbar force-pushed the add-roles branch 4 times, most recently from 34e1377 to 6617969 Compare June 11, 2026 09:50
@micbar
micbar requested review from fschade and kulmann June 11, 2026 09:50
Comment thread pkg/conversions/role.go
return NewSpaceEditorRole()
case RoleFileEditor:
return NewFileEditorRole()
case RoleFileEditorWithVersions:

@kulmann kulmann Jun 11, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why don't we need a file viewer with versions role?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

viewer applies for all resources.

@micbar
micbar merged commit d03d0fa into main Jun 11, 2026
19 checks passed
@micbar
micbar deleted the add-roles branch June 11, 2026 12:34
@openclouders openclouders mentioned this pull request Jun 11, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants