-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Lens] allow top metric for last value #127151
[Lens] allow top metric for last value #127151
Conversation
@elasticmachine merge upstream |
…ndrewctate/kibana into 102956/allow-top-metric-for-last-value
Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors) |
…ndrewctate/kibana into 102956/allow-top-metric-for-last-value
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.
Looks pretty good already, left some comments
x-pack/plugins/lens/public/indexpattern_datasource/operations/definitions/terms/helpers.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/lens/public/indexpattern_datasource/operations/definitions/last_value.tsx
Outdated
Show resolved
Hide resolved
x-pack/plugins/lens/public/indexpattern_datasource/operations/definitions/last_value.tsx
Outdated
Show resolved
Hide resolved
…p-metric-for-last-value
Can use this exported dashboard to test the migration for both by-value and by-reference tables. Just add the e-commerce sample data first. |
@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.
Tested and works as expected, LGTM with one minor nit.
x-pack/plugins/lens/public/indexpattern_datasource/operations/definitions/last_value.tsx
Show resolved
Hide resolved
…ndrewctate/kibana into 102956/allow-top-metric-for-last-value
…p-metric-for-last-value
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💛 Build succeeded, but was flakyMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
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.
Changes in observability LGTM
Summary
Top metrics for last value
Resolves #102956
Checklist
Delete any items that are not applicable to this PR.