Skip to content

[8.19] [Security Solution] Remove null in confirmation dialog when bulk editing index patterns for rules. (#236572)#237091

Merged
kibanamachine merged 2 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-236572
Oct 1, 2025
Merged

[8.19] [Security Solution] Remove null in confirmation dialog when bulk editing index patterns for rules. (#236572)#237091
kibanamachine merged 2 commits intoelastic:8.19from
kibanamachine:backport/8.19/pr-236572

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…iting index patterns for rules. (elastic#236572)

**Resolves:  elastic#235794**

## Summary

This PR removes a `null` value appearing on a success confirmation
dialog when bulk-editing multiple security rules and setting a new index
pattern.

<img width="500"
src="https://github.com/user-attachments/assets/06241840-3bc0-4eec-93b0-94755c77cd9d"
/>

This is done by swapping `null` for empty string and adding unit tests
for `explainBulkExitSucess()`.

### Testing

To test this PR please checkout the relevant branch and run kibana
locally.

1. Security App > `Rules` > `Detection Rules (SIEM)`
2. Make sure several rules are created or pre installed.
3. Select 3 rules > `Bulk actions` > `Index patterns` > `Add Index
patterns`
4. Add index pattern (ie test123-*) > `Save`

Expected Result:

5. Dialog shows: `Rules Updated: You've successfully updated 3 rules.`

<details>
<summary><h3>Click to see detailed testing steps
(screenshots)</h3></summary>

<img width="800"
src="https://github.com/user-attachments/assets/b5cdb962-7e01-4cec-8021-0ff3928b2615"
/>

<img width="800"
src="https://github.com/user-attachments/assets/07692af9-7f61-499a-b813-359aeee6def7"
/>

<img width="800"
src="https://github.com/user-attachments/assets/9332be94-41ab-4325-81d6-acf7ee947fb0"
/>

</details>

(cherry picked from commit dff0cac)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Oct 1, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) October 1, 2025 10:13
@kibanamachine kibanamachine merged commit c4bee44 into elastic:8.19 Oct 1, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

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.5MB 9.5MB -2.0B

History

cc @sdesalas

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