-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[ML] Explain Log Rate Spikes: Adds discover link to analysis table #139877
[ML] Explain Log Rate Spikes: Adds discover link to analysis table #139877
Conversation
Looking at the screenshot, suggest to add a fixed width to the actions column to make it more narrow. |
Pinging @elastic/ml-ui (:ml) |
x-pack/plugins/aiops/public/components/spike_analysis_table/spike_analysis_table.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/aiops/public/components/spike_analysis_table/links_menu.tsx
Outdated
Show resolved
Hide resolved
…-ref HEAD~1..HEAD --fix'
enabled: () => discoverUrlError === undefined, | ||
}, | ||
], | ||
width: '60px', |
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.
Nit: For the other columns we already have a constant with the custom width (NARROW_COLUMN_WIDTH
). Even if this is just one other column, suggest to create another constant like ACTION_COLUMN_WIDTH
for this.
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.
Updated in 344f7b0
x-pack/plugins/aiops/public/components/spike_analysis_table/spike_analysis_table.tsx
Outdated
Show resolved
Hide resolved
@elasticmachine merge upstream |
This has been updated with suggestions and is ready for a final look |
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.
Code LGTM, just need to remove one commented line
x-pack/plugins/aiops/public/components/spike_analysis_table/spike_analysis_table.tsx
Outdated
Show resolved
Hide resolved
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.
Tested latest changes and LGTM
…ike_analysis_table.tsx Co-authored-by: Dima Arnautov <[email protected]>
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
Summary
Related meta issue: #138117
This PR adds an Actions column to the Spike Analysis Table with an item to link to discover for that key/value pair.
Checklist
Delete any items that are not applicable to this PR.