Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[RAM} Add refine search prompt to rule execution log #128982

Merged

Conversation

JiaweiWu
Copy link
Contributor

@JiaweiWu JiaweiWu commented Mar 30, 2022

Summary

Resolves: #128811

Adds a prompt to notify the use that they should refine their search if it is too broad. Similar to how it is done in the discover plugin. The current threshold is 500 as discussed here: #127339 (comment)

Screenshot from 2022-03-30 13-26-18

@JiaweiWu JiaweiWu added release_note:enhancement Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) auto-backport Deprecated - use backport:version if exact versions are needed v8.2.0 labels Mar 30, 2022
@JiaweiWu JiaweiWu requested a review from XavierM March 30, 2022 20:29
@JiaweiWu
Copy link
Contributor Author

@elasticmachine merge upstream

@JiaweiWu
Copy link
Contributor Author

JiaweiWu commented Apr 4, 2022

@elasticmachine merge upstream

@JiaweiWu JiaweiWu added release_note:skip Skip the PR/issue when compiling release notes v8.3.0 and removed release_note:enhancement labels Apr 4, 2022
@JiaweiWu JiaweiWu marked this pull request as ready for review April 4, 2022 17:07
@JiaweiWu JiaweiWu requested a review from a team as a code owner April 4, 2022 17:07
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Test Failures

  • [job] [logs] OSS Firefox Tests / discover app discover tab field data shows top-level object keys

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
triggersActionsUi 359 360 +1

Async chunks

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

id before after diff
triggersActionsUi 697.7KB 699.6KB +1.9KB

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@@ -256,6 +257,10 @@ export const RuleErrorLog = (props: RuleErrorLogProps) => {
}
}}
/>
<RefineSearchPrompt
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@JiaweiWu JiaweiWu merged commit 2fd6772 into elastic:main Apr 5, 2022
kibanamachine pushed a commit that referenced this pull request Apr 5, 2022
* Refine search prompt

* Add refine prompt to error log table

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit 2fd6772)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.2

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Apr 5, 2022
* Refine search prompt

* Add refine prompt to error log table

Co-authored-by: Kibana Machine <[email protected]>
(cherry picked from commit 2fd6772)

Co-authored-by: Jiawei Wu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.2.0 v8.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[RAM] Add prompt to refine search to the rule execution log list
5 participants