Skip to content

[8.x] [Security Solution] Adds check for license level for rule editing callout message (#213995)#214198

Merged
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-213995
Mar 12, 2025
Merged

[8.x] [Security Solution] Adds check for license level for rule editing callout message (#213995)#214198
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-213995

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…lout message (elastic#213995)

## Summary

Resolves elastic#213983

Adds gating logic to only display the rule upgrade related warning
callout on the rule editing page if the user has the appropriate license
level, otherwise the message is irrelevant as the user wouldn't be able
to edit a prebuilt rule in the way the message refers to

### Testing

**Steps to reproduce:**

1. Use a Platinum license or any tier that does not allow prebuilt rule
updates.
2. Navigate to the Rule Management page and select a prebuilt rule with
an available update.
3. Open Rule Details page -> Edit rule settings
4. Observe the callout message is not displayed at the top of the page

NOTE: When using the enterprise license, the callout should still be
displayed

(cherry picked from commit 0485578)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Mar 12, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) March 12, 2025 15:09
@kibanamachine kibanamachine merged commit 17fb712 into elastic:8.x Mar 12, 2025
11 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 9.1MB 9.1MB +3.0B

cc @dplumlee

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants