-
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
[Logs UI] Fix log rate table row expansion #60096
[Logs UI] Fix log rate table row expansion #60096
Conversation
Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui) |
@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.
Fix LGTM, but why does Item
extend from any
here?
kibana/x-pack/plugins/infra/public/components/basic_table/row_expansion_button.tsx
Line 11 in 33c631d
export const RowExpansionButton = <Item extends any>({ |
That seems like the root of the bug not being caught, yeah?
Well, it's a generic component because it's used in different circumstances. The real cause is the not being typed 🤔 I'll try to reformulate to avoid that, thanks for the question. |
@weltenwort sounds good, I was more suggesting maybe not to extend from anything and force users to specify the item's type. |
But I forgot that extends doesn't work as a default, so we are already forcing that. 👍 |
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / kibana-xpack-agent / X-Pack Alerting API Integration Tests.x-pack/test/alerting_api_integration/security_and_spaces/tests/alerting/alerts·ts.alerting api integration security and spaces enabled Alerting alerts superuser at space1 should schedule task, run alert and schedule actions when appropriateStandard Out
Stack Trace
History
To update your PR or re-run it, just comment with: |
This fixes the log rate table row expansion button, which broke in elastic#54586 during a refactoring.
Summary
This fixes the log rate table row expansion button, which I broke in #54586.
Testing
before: Clicking the expansion button didn't have any effect.
after: Clicking the expansion button expands the row as expected.