[Logs UI] Fix log rate table row expansion#60096
Conversation
|
Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui) |
|
@elasticmachine merge upstream |
jasonrhodes
left a comment
There was a problem hiding this comment.
Fix LGTM, but why does Item extend from any here?
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 OutStack TraceHistory
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.