Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Include icon in RemoteApplicationInfo result #6766
Conversation
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
jujubot
merged commit 805d59a
into
juju:develop
Jan 5, 2017
1 check failed
github-check-merge-juju
Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
wallyworld commentedJan 5, 2017
The RemoteApplicationInfo api now includes the icon for remote apps from another model in the same controller.
Some code in apiserver/charms was refactored to extract common functionality that is now used by both the charm http handler as well as the application facade. A drive by fix was also done to better locate a helper newStorage method in the file which uses it.
QA: bootstrap and check GUI renders correctly.