Skip to content

[7.x] [NP] Inline buildPointSeriesData and buildHierarchicalData dependencies (#61575)#63145

Merged
maryia-lapata merged 1 commit intoelastic:7.xfrom
maryia-lapata:backport/7.x/pr-61575
Apr 10, 2020
Merged

[7.x] [NP] Inline buildPointSeriesData and buildHierarchicalData dependencies (#61575)#63145
maryia-lapata merged 1 commit intoelastic:7.xfrom
maryia-lapata:backport/7.x/pr-61575

Conversation

@maryia-lapata
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…es (elastic#61575)

* Move buildHierarchicalData to vislib

* Move shortened version of buildPointSeriesData to Discover

* Move buildPointSeriesData to vis_type_vislib

* Convert unit tests to jest

* Remove ui/agg_response

* Convert point_series files to TS

* Update TS in unit tests

* Convert buildHierarchicalData to TS

* Convert buildPointSeriesData to TS in Discover

* Clean TS in Discover

* Update TS for buildHierarchicalData

* Update buildHierarchicalData unit tests

* Clean up TS in point_series

* Add unit tests fro response_handler.js

* Simplify point_series for Discover

* Return array for data

* Add check for empty row

* Simplify point_series for Discover

* Return all points

* Specify TS

* Refactoring

* Simplifying

* improve types

* Update _get_point.test.ts

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: Joe Reuter <johannes.reuter@elastic.co>
@maryia-lapata maryia-lapata added the backport This PR is a backport of another PR label Apr 9, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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

@maryia-lapata maryia-lapata merged commit 7b0c852 into elastic:7.x Apr 10, 2020
@maryia-lapata maryia-lapata deleted the backport/7.x/pr-61575 branch April 10, 2020 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants