Skip to content

[8.x] [Response Ops] Es query rule "size" field re-initializes to 100 if set to 0 when editing (#213636)#214848

Merged
doakalexi merged 3 commits intoelastic:8.xfrom
doakalexi:backport/8.x/pr-213636
Mar 18, 2025
Merged

[8.x] [Response Ops] Es query rule "size" field re-initializes to 100 if set to 0 when editing (#213636)#214848
doakalexi merged 3 commits intoelastic:8.xfrom
doakalexi:backport/8.x/pr-213636

Conversation

@doakalexi
Copy link
Contributor

Backport

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

Questions ?

Please refer to the Backport tool documentation

…t to 0 when editing (elastic#213636)

Resolves elastic#209427

## Summary

This PR fixes a bug when editing an es query rule with size set to 0. I
also refactored the tests to use react testing library.

### Checklist

Check the PR satisfies following conditions.

- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

### To verify

1. Create an es query rule
2. Set the size to be 0
3. Save your rule
4. Edit your rule and verify that the size is set to 0 when you open the
editor

(cherry picked from commit 287eb3e)
@doakalexi doakalexi added the backport This PR is a backport of another PR label Mar 17, 2025
@doakalexi doakalexi enabled auto-merge (squash) March 17, 2025 17:47
@elasticmachine
Copy link
Contributor

elasticmachine commented Mar 18, 2025

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

‼️ ERROR: no builds found for mergeBase sha [9ad3f73]

History

@doakalexi doakalexi merged commit db7277d into elastic:8.x Mar 18, 2025
8 checks passed
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