Skip to content

[6.x] [core/public/chrome] migrate controls, theme, and visibility apis (#22987)#24308

Merged
spalger merged 1 commit intoelastic:6.xfrom
spalger:backport/6.x/pr-22987
Oct 20, 2018
Merged

[6.x] [core/public/chrome] migrate controls, theme, and visibility apis (#22987)#24308
spalger merged 1 commit intoelastic:6.xfrom
spalger:backport/6.x/pr-22987

Conversation

@spalger
Copy link
Copy Markdown
Contributor

@spalger spalger commented Oct 19, 2018

Backports the following commits to 6.x:

…astic#22987)

* [core/public/chrome] migrate controls, theme, and visibility apis

* [core/public] stop uiSettings service

* [core/public/chrome] test that observables stop immedaiately after stop()

* fix typos

* [core/public/legacyPlatform] test globalNavState init

* [ui/chrome] don't pass extra params

* [core/public/chrome] test for dedupe-handling

* [ui/chrome/theme] test with different values for logo and smallLogo
@spalger spalger added the backport This PR is a backport of another PR label Oct 19, 2018
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@spalger spalger merged commit e33cba3 into elastic:6.x Oct 20, 2018
@spalger spalger deleted the backport/6.x/pr-22987 branch October 22, 2018 16:00
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