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
[Security Solution] Display job friendly name on detection pages #148974
Conversation
8519b41
to
a1aced6
Compare
a1aced6
to
86e99c9
Compare
@elasticmachine merge upstream |
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pulled in and tested locally, and everything works as expected. Thank you, @machadoum 👍
This will also resolve #147722 that I opened a little while back @machadoum 🙂 |
@elasticmachine merge upstream |
79d1c3f
to
92e9057
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@elasticmachine merge upstream |
After updating the UI to display the job name, we lost the search by job id feature. Because some users might be used to search by id, this commit reintroduces the feature.
81d2b30
to
af3c52d
Compare
💛 Build succeeded, but was flaky
Failed CI StepsMetrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @machadoum |
…stic#148974) issue: elastic#148973 Similar to: elastic#146772 ## Summary When friendly names are available as job meta information, we should display it on: * Manage Rules table - Job warning message * New rule page - Machine Learning job selector * Rule details page - Machine Learning job description and warning message <img width="600" alt="Screenshot 2023-01-16 at 14 34 50" src="https://user-images.githubusercontent.com/1490444/212692378-9fcb59b5-179c-4054-b2b8-9972b9f35bab.png"> <img width="600" alt="Screenshot 2023-01-16 at 14 33 14" src="https://user-images.githubusercontent.com/1490444/212692408-bc978eea-a4c6-4a43-bb67-3098969da7e2.png"> <img width="600" alt="Screenshot 2023-01-16 at 13 39 51" src="https://user-images.githubusercontent.com/1490444/212692445-8bacb22d-5160-45f7-aa56-ec5f296d3fb4.png"> ### Checklist - [x] [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
issue: #148973
Similar to: #146772
Summary
When friendly names are available as job meta information, we should display it on:
Checklist