Skip to content

Fetch options for input control child select boxes - #38148

Merged
nreese merged 1 commit into
elastic:masterfrom
nreese:issue_38128
Jun 7, 2019
Merged

nreese merged 1 commit into
elastic:masterfrom
nreese:issue_38128

Conversation

@nreese

@nreese nreese commented Jun 5, 2019 •

Copy link
Copy Markdown
Contributor

fixes #38128

The actual fix is only a few changes in list_control_factory.

The rest of the changes resulted in the input control functional tests needed some refactoring.

  1. The tests were in a single suite that took forever to run. So this PR breaks them into 3 separate suites.
  2. This PR also loads saved objects for dynamic and chained tests to avoid the pain of having to use the UI to build the test artifacts.
  3. The PR switched data.json.gz to just data.json so its readable and changes can be tracked easier.
  4. added a test case for chained controls and dynamic option filtering

@nreese nreese added release_note:fix Feature:Input Control Input controls visualization Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v8.0.0 v7.2.0 v7.3.0 labels Jun 5, 2019
@nreese
nreese requested a review from timroes June 5, 2019 18:44
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-app

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@flash1293
flash1293 self-requested a review June 7, 2019 08:57

@timroes timroes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code LGTM, tested on Chrome Linux, requests are now also made for child components.

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

Labels

Feature:Input Control Input controls visualization release_note:fix Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.2.0 v7.3.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Input controls child elements are not refetching data

3 participants