Skip to content

[7.x] [Vis] TableVis uses local angular (#50759)#51962

Merged
maryia-lapata merged 4 commits intoelastic:7.xfrom
maryia-lapata:backport/7.x/pr-50759
Dec 3, 2019
Merged

[7.x] [Vis] TableVis uses local angular (#50759)#51962
maryia-lapata merged 4 commits intoelastic:7.xfrom
maryia-lapata:backport/7.x/pr-50759

Conversation

@maryia-lapata
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* TableVis uses local angular

* Clean up

* Fix TS

* Update angular_config

* Fix export

* Update render_app.ts

* Cetralize ui deps

* Fix loading KbnTableVisController in Dashboard

* Fix graph

* Rename const

* Add table vis mocks

* Fix kbn_top_nav

* Add TS for test

* Complete conversion paginated_table test to Jest

* Convert table_vis_controller test to Jest

* Convert table_vis_controller test to Jest

* Create agg_table.test.ts

* Fix mocha tests

* Refactoring

* Remove module dep

* Remove LegacyDependenciesPlugin

* Move file

* Fix path

* Fix path

* Fix TS

* Fix Jest test
@maryia-lapata maryia-lapata added the backport This PR is a backport of another PR label Dec 2, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@maryia-lapata
Copy link
Copy Markdown
Contributor Author

This PR should be backported after #51947.

@elasticmachine
Copy link
Copy Markdown
Contributor

💔 Build Failed

@maryia-lapata
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@maryia-lapata maryia-lapata merged commit 83fb4f6 into elastic:7.x Dec 3, 2019
@maryia-lapata maryia-lapata deleted the backport/7.x/pr-50759 branch December 3, 2019 09:33
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