Skip to content

[8.x] [Security Solution] Fix incorrect default value for History Window Size (#207827)#208178

Merged
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-207827
Jan 24, 2025
Merged

[8.x] [Security Solution] Fix incorrect default value for History Window Size (#207827)#208178
kibanamachine merged 1 commit intoelastic:8.xfrom
kibanamachine:backport/8.x/pr-207827

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

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

Questions ?

Please refer to the Backport tool documentation

…ze (elastic#207827)

**Resolves: elastic#207348

## Summary
This PR resolves a bug in the editable History Window Size component,
which incorrectly displays "0 seconds" as the default value instead of
"7 days". The issue was caused by passing an array of units `(['m', 'h',
'd'])` using an incorrect prop name.

Also, I fixed stuck last digit being stuck when trying to remove it
**Before my changes: can't remove or reset the last digit**

https://github.com/user-attachments/assets/3a7d28ea-ea71-4ee8-8805-902bae1dc3c1

**After my changes: value resets to minValue on last digit removal**

https://github.com/user-attachments/assets/1cba36bb-9127-4197-871e-62b978b3612c

Work started on: 22-Jan-2025

(cherry picked from commit 8894c1f)
@kibanamachine kibanamachine merged commit 3e02341 into elastic:8.x Jan 24, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #39 / Data Views feature controls security no data views privileges "after all" hook: afterTestSuite.trigger for "doesn't show Data Views in management side-nav"

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 18.5MB 18.5MB +18.0B

cc @nikitaindik

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