Skip to content

Add autocomplete to TSVB - #30158

Merged
lukasolson merged 3 commits into
elastic:masterfrom
lukasolson:tsvbAutocomplete
Feb 7, 2019
Merged

lukasolson merged 3 commits into
elastic:masterfrom
lukasolson:tsvbAutocomplete

Conversation

@lukasolson

Copy link
Copy Markdown
Contributor

Adds autocomplete to TSVB on dashboards.

This is the first phase as it only autocompletes on the index pattern you've selected for your panel data, not custom series index patterns or annotations. (This will require rewiring some things to allow multiple index patterns.)

@lukasolson lukasolson added review Feature:TSVB TSVB (Time Series Visual Builder) v7.0.0 Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// Feature:KQL KQL labels Feb 5, 2019
@lukasolson lukasolson self-assigned this Feb 5, 2019
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-app

@elasticmachine

Copy link
Copy Markdown
Contributor

💔 Build Failed

@rayafratkina rayafratkina mentioned this pull request Feb 6, 2019
@lukasolson

Copy link
Copy Markdown
Contributor Author

Retest

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@Bargs Bargs 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.

LGTM

@lukasolson
lukasolson merged commit d625087 into elastic:master Feb 7, 2019
lukasolson added a commit that referenced this pull request Feb 7, 2019
* Add autocomplete to TSVB

* Remove unused imports

* Remove unnecessary ts-ignore
lukasolson added a commit that referenced this pull request Feb 7, 2019
* Add autocomplete to TSVB

* Remove unused imports

* Remove unnecessary ts-ignore
@lukasolson

Copy link
Copy Markdown
Contributor Author

7.x (7.1.0): 885633e
7.0 (7.0.0): 729ccf4

import { getFromSavedObject } from 'ui/index_patterns/static_utils';
import { VisSavedObject } from 'ui/visualize/loader/types';

export async function getIndexPattern(

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What was the reasoning behind putting this function in the embeddable folder? It doesn't seem to be tied to embeddables, and relies on specific visualization saved object knowledge.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You're right, this should be moved to public/visualize somewhere

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Do you mind pulling together a follow up PR for this move?

@lukasolson
lukasolson deleted the tsvbAutocomplete branch December 2, 2019 22:34
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
* Add autocomplete to TSVB

* Remove unused imports

* Remove unnecessary ts-ignore
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:KQL KQL Feature:TSVB TSVB (Time Series Visual Builder) review Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.0.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants