Skip to content

[Lens] Fix bug in metric config panel#60982

Merged
wylieconlon merged 3 commits intoelastic:masterfrom
wylieconlon:lens/fix-metric-accessor
Mar 24, 2020
Merged

[Lens] Fix bug in metric config panel#60982
wylieconlon merged 3 commits intoelastic:masterfrom
wylieconlon:lens/fix-metric-accessor

Conversation

@wylieconlon
Copy link
Contributor

Summary

Fixes bug where you could not set a metric without using suggestions.

Checklist

@wylieconlon wylieconlon added Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Feature:Lens v7.7.0 labels Mar 23, 2020
@wylieconlon wylieconlon requested review from a team, mbondyra and timroes March 23, 2020 19:53
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@mbondyra
Copy link
Contributor

Hi Wylie, can you please write a quick note how I can reproduce this bug?

@wylieconlon
Copy link
Contributor Author

Before this fix, the bug was reproduced by:

  1. Going to the metric visualization in an empty state
  2. Not being allowed to do anything else without suggestions

Copy link
Contributor

@mbondyra mbondyra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested on chrome, LGTM 🆗

@wylieconlon wylieconlon merged commit 6d2aa89 into elastic:master Mar 24, 2020
@wylieconlon wylieconlon deleted the lens/fix-metric-accessor branch March 24, 2020 14:43
gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 24, 2020
* master:
  Updating our direct usage of https-proxy-agent to 5.0.0 (elastic#58296)
  allow users to unset the throttle of an alert (elastic#60964)
  [Lens] Fix bug in metric config panel (elastic#60982)
  [SearchProfiler] Minor fixes (elastic#60919)
  [ML] Renaming ML setup and start contracts (elastic#60980)
  introduce StartServicesAccessor type for `CoreSetup.getStartServices` (elastic#60748)
  [SIEM][Detection Engine] Add rule's notification alert type (elastic#60832)
  [APM] Re-revert "Collect telemetry about data/API performance" (elastic#61030)
  [NP] Graph: get rid of saved objects class wrapper (elastic#59917)
  [EPM] merge duplicate fields when creating index patterns (elastic#60957)
  [Uptime] Ml detection of duration anomalies (elastic#59785)
  [Alerting] removes unimplemented buttons from Alert Details page (elastic#60934)
  [skip-ci] Fix CODEOWNERS paths for the Pulse team (elastic#60944)
  [APM] Threshold alerts (elastic#59566)
  [ML] Add support for percentiles aggregation to Transform wizard (elastic#60763)
  Cahgen save object duplicate message (elastic#60901)
wylieconlon pushed a commit that referenced this pull request Mar 24, 2020
* [Lens] Fix bug in metric config panel

* Fix test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Lens release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.7.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants