Skip to content

Comments

[7.x] [Dashboard] Add visualization by value to dashboard (#69898)#70317

Closed
majagrubic wants to merge 1 commit intoelastic:7.xfrom
majagrubic:backport/7.x/pr-69898
Closed

[7.x] [Dashboard] Add visualization by value to dashboard (#69898)#70317
majagrubic wants to merge 1 commit intoelastic:7.xfrom
majagrubic:backport/7.x/pr-69898

Conversation

@majagrubic
Copy link
Contributor

Backports the following commits to 7.x:

* Plugging in DashboardStart dependency

* Create embeddable by reference and navigate back to dashboard

* Trying to feature flag the new flow

* Feature flagging new visualize flow

* Removing unnecessary console statement

* Fixing typescript errors

* Adding a functional test for new functionality

* Adding a functional test for new functionality

* Fixing test name

* Changing test name

* Moving functional test to a separate folder

* Trying to fix the config file

* Adding an index file

* Remove falsly included file

* Adding aggs and params to vis input

* Serializing vis before passing it as an input

* Incorporating new state transfer logic

* Remove dashboardStart as a dependency

* Trying to get the test to run

* Remove unused import

* Readding spaces

* Fixing type errors

* Incorporating new changes
# Conflicts:
#	scripts/functional_tests.js
#	tasks/function_test_groups.js
@majagrubic majagrubic added the backport This PR is a backport of another PR label Jun 30, 2020
@majagrubic majagrubic requested a review from dmlemeshko June 30, 2020 13:51
@majagrubic majagrubic closed this Jun 30, 2020
@majagrubic majagrubic deleted the backport/7.x/pr-69898 branch June 30, 2020 13:58
@kibanamachine
Copy link
Contributor

💔 Build Failed

Failed CI Steps

Build metrics

@kbn/optimizer bundle module count

id value diff baseline
visualize 184 +5 179

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

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