Skip to content

[5.x] Streamline adding a visualization to a dashboard (#9400)#9756

Merged
stacey-gammon merged 1 commit intoelastic:5.xfrom
stacey-gammon:5.x
Jan 6, 2017
Merged

[5.x] Streamline adding a visualization to a dashboard (#9400)#9756
stacey-gammon merged 1 commit intoelastic:5.xfrom
stacey-gammon:5.x

Conversation

@stacey-gammon
Copy link
Copy Markdown

Backports #9400

  • Fix bug with param sticking around in the url causing multiple vis to add again and again

Move the link into the saved object finder

and get rid of the management link
fix crash in tests
lint fixes

  • rebase with master

  • code review comments

make the function simpler and reduce the need to pass null everywhere

  • address comments and cleanup

* Fix bug with param sticking around in the url causing multiple vis to add again and again

Move the link into the saved object finder

and get rid of the management link
fix crash in tests
lint fixes

* rebase with master

* code review comments

make the function simpler and reduce the need to pass null everywhere

* address comments and cleanup
@stacey-gammon stacey-gammon merged commit daabb70 into elastic:5.x Jan 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant