Skip to content

Comments

[7.x] [ML] Transforms: Use EuiInMemoryTable instead of custom typed table. (#59782)#59868

Merged
walterra merged 5 commits intoelastic:7.xfrom
walterra:backport/7.x/pr-59782
Mar 11, 2020
Merged

[7.x] [ML] Transforms: Use EuiInMemoryTable instead of custom typed table. (#59782)#59868
walterra merged 5 commits intoelastic:7.xfrom
walterra:backport/7.x/pr-59782

Conversation

@walterra
Copy link
Contributor

Backports the following commits to 7.x:

…lastic#59782)

Before EuiInMemoryTable had TypeScript support we used our own typings and some state & CSS fixes. This is now all solved by the original EUI component.
- Use EuiInMemoryTable instead of custom typed table.
- Deletes some legacy leftover files.
@walterra walterra added the backport This PR is a backport of another PR label Mar 11, 2020
@walterra walterra self-assigned this Mar 11, 2020
@walterra walterra added the :ml label Mar 11, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@walterra
Copy link
Contributor Author

@elasticmachine merge upstream

@walterra
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@walterra walterra merged commit fad00d9 into elastic:7.x Mar 11, 2020
@walterra walterra deleted the backport/7.x/pr-59782 branch March 11, 2020 16:47
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