Skip to content

Timelion reads raw field list from saved object, should grab from index pattern api#89988

Merged
alexwizp merged 2 commits intoelastic:masterfrom
alexwizp:84387
Feb 3, 2021
Merged

Timelion reads raw field list from saved object, should grab from index pattern api#89988
alexwizp merged 2 commits intoelastic:masterfrom
alexwizp:84387

Conversation

@alexwizp
Copy link
Copy Markdown
Contributor

@alexwizp alexwizp commented Feb 2, 2021

Closes: #84387

Summary

Timelion reads the index pattern field list from the index pattern saved object. Instead use the index pattern service to load the index pattern.

Dev/Testing notes

savedObjectsClient was changed to indexPatterns service in following places:

  • arg_value_suggestions.ts
    To test suggestions please you can use the following timeline expressions.
    Index suggestions:
    image
    Fields suggestions:
    image

  • series_functions/es/index.js
    Scripted fields suggestions:
    image

@elastic elastic deleted a comment from kibanamachine Feb 2, 2021
@elastic elastic deleted a comment from kibanamachine Feb 2, 2021
@alexwizp alexwizp self-assigned this Feb 2, 2021
@alexwizp alexwizp added v7.12.0 v8.0.0 Feature:Timelion Timelion app and visualization 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// labels Feb 2, 2021
@alexwizp alexwizp marked this pull request as ready for review February 2, 2021 12:14
@alexwizp alexwizp requested a review from a team February 2, 2021 12:14
@elasticmachine
Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

@VladLasitsa VladLasitsa left a comment

Choose a reason for hiding this comment

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

LGTM

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
visTypeTimelion 25.7KB 24.8KB -927.0B

History

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

Copy link
Copy Markdown
Contributor

@stratoula stratoula left a comment

Choose a reason for hiding this comment

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

LGTM! Thanx @alexwizp for that 🙂

@alexwizp alexwizp merged commit af454d6 into elastic:master Feb 3, 2021
alexwizp added a commit to alexwizp/kibana that referenced this pull request Feb 3, 2021
…ex pattern api (elastic#89988)

* Timelion reads raw field list from saved object, should grab from index pattern api

Closes: elastic#84387

* fix PR comment
alexwizp added a commit that referenced this pull request Feb 3, 2021
…ex pattern api (#89988) (#90124)

* Timelion reads raw field list from saved object, should grab from index pattern api

Closes: #84387

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

Labels

Feature:Timelion Timelion app and visualization 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.12.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Timelion] Timelion reads raw field list from saved object, should grab from index pattern api

5 participants