Skip to content

[7.x] [ML] EuiDataGrid ml/transform components. (#63447)#64417

Merged
walterra merged 1 commit intoelastic:7.xfrom
walterra:backport/7.x/pr-63447
Apr 24, 2020
Merged

[7.x] [ML] EuiDataGrid ml/transform components. (#63447)#64417
walterra merged 1 commit intoelastic:7.xfrom
walterra:backport/7.x/pr-63447

Conversation

@walterra
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

Cleanup and consolidation of code related to EuiDataGrid. The transform wizard source and pivot preview table as well as the data frame analytics results pages now share a common code base related to data grid tables.

To avoid tight coupling of components and hooks, the hooks are not within the common data grid component. Instead the hooks need to be used on the outer wrapping component and the results will be passed as props to the data grid component. This allows us to pass data from different data sources (transform index source, pivot previews, analytics results) into a shared component.
@walterra walterra added the backport This PR is a backport of another PR label Apr 24, 2020
@walterra walterra self-assigned this Apr 24, 2020
@walterra walterra added the :ml label Apr 24, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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

@walterra walterra merged commit 19c2a2a into elastic:7.x Apr 24, 2020
@walterra walterra deleted the backport/7.x/pr-63447 branch April 24, 2020 18:59
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 :ml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants