Skip to content

Fix hiding the <CreateButton /> on overview pages if the user lags permissions#26731

Merged
kmerz merged 1 commit into
masterfrom
fix/issue-14849
Jul 24, 2026
Merged

Fix hiding the <CreateButton /> on overview pages if the user lags permissions#26731
kmerz merged 1 commit into
masterfrom
fix/issue-14849

Conversation

@kmerz

@kmerz kmerz commented Jul 22, 2026

Copy link
Copy Markdown
Member

Why

'Feat/14047 sigma rules to event definition wizard UI' moved the code to handle permissions out side of the component to treat it seperatly.

With that the code no longer had access to the entityCreator.permissions and the component was renderded without permissions. That way the component was always rendered nevertheless the user did not have the right permissions.

What

We introduced a simple test which ensures that the button is hided if a user does not have the right permissions.

We pass the entityCreator.permissions to the new

Fixes https://github.com/Graylog2/graylog-plugin-enterprise/issues/14849

/nocl

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have requested a documentation update.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

…rmissions

## Why
'Feat/14047 sigma rules to event definition wizard UI' moved the
code to handle permissions out side of the component to treat it
seperatly.

With that the code no longer had access to the entityCreator.permissions
and the component was renderded without permissions.
That way the <CreateButton /> component was always rendered
nevertheless the user did not have the right permissions.

## What
We introduced a simple test which ensures that the button is hided
if a user does not have the right permissions.

We pass the entityCreator.permissions to the new <PermissionsWrapper />
@kmerz
kmerz marked this pull request as ready for review July 23, 2026 09:35
@kmerz
kmerz requested a review from zeeklop July 23, 2026 09:36
@kmerz
kmerz merged commit 6d9e7bd into master Jul 24, 2026
27 of 28 checks passed
@kmerz
kmerz deleted the fix/issue-14849 branch July 24, 2026 11:59
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