-
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] Formula can be colored by value #103572
[Lens] Formula can be colored by value #103572
Conversation
Pinging @elastic/kibana-app-services (Team:AppServices) |
Pinging @elastic/kibana-app (Team:KibanaApp) |
This error is great:
It's working better than before and is applying the number formatter correctly now |
💚 Build SucceededMetrics [docs]Page load bundle
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.
LGTM
reviewed the code and ran the functional test locally
* [Lens] Formula can be colored by value * Fix mapColumn test * Fix tests
💔 Backport failed
Successful backport PRs will be merged automatically after passing CI. To backport manually run: |
* [Lens] Formula can be colored by value * Fix mapColumn test * Fix tests Co-authored-by: Wylie Conlon <[email protected]>
* [Lens] Formula can be colored by value * Fix mapColumn test * Fix tests
* [Lens] Formula can be colored by value * Fix mapColumn test * Fix tests Co-authored-by: Wylie Conlon <[email protected]>
Fixes bug where the
mathColumn
was returning anull
type when the first row was null, which is guaranteed when usingdifferences
in formulas.Checklist