Skip to content

feat: [OCISDEV-527] check vault permission#13802

Merged
LukasHirt merged 1 commit into
feat/OCISDEV-524/mode-switchfrom
feat/vault-permissions
May 21, 2026
Merged

feat: [OCISDEV-527] check vault permission#13802
LukasHirt merged 1 commit into
feat/OCISDEV-524/mode-switchfrom
feat/vault-permissions

Conversation

@LukasHirt
Copy link
Copy Markdown
Collaborator

Description

When the user has the VaultMode.ReadWriteEnabled.own permission, the mode switch will be shown in the topbar.

Related Issue

Motivation and Context

Users who hasn't got permissions to access vault should not see the mode switcher so that they do not try to access it.

How Has This Been Tested?

  • test environment: macos, chrome
  • test case 1: open web with admin user
  • test case 2: open web with regular user
  • test case 3: open web with light user

When the user has the `VaultMode.ReadWriteEnabled.own` permission,
the mode switch will be shown in the topbar.

Signed-off-by: Lukas Hirt <info@hirt.cz>
@LukasHirt LukasHirt requested a review from mzner May 21, 2026 12:09
@LukasHirt LukasHirt self-assigned this May 21, 2026
@LukasHirt LukasHirt merged commit 04d9f95 into feat/OCISDEV-524/mode-switch May 21, 2026
1 check passed
@LukasHirt LukasHirt deleted the feat/vault-permissions branch May 21, 2026 12:19
mzner pushed a commit that referenced this pull request May 22, 2026
When the user has the `VaultMode.ReadWriteEnabled.own` permission,
the mode switch will be shown in the topbar.

Signed-off-by: Lukas Hirt <info@hirt.cz>
mzner pushed a commit that referenced this pull request May 22, 2026
When the user has the `VaultMode.ReadWriteEnabled.own` permission,
the mode switch will be shown in the topbar.

Signed-off-by: Lukas Hirt <info@hirt.cz>
mzner pushed a commit that referenced this pull request May 22, 2026
When the user has the `VaultMode.ReadWriteEnabled.own` permission,
the mode switch will be shown in the topbar.

Signed-off-by: Lukas Hirt <info@hirt.cz>
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.

2 participants