Merge lp:~vthompson/music-app/fix-1478162 into lp:music-app
Proposed by
Victor Thompson
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Andrew Hayzen | ||||
Approved revision: | 890 | ||||
Merged at revision: | 889 | ||||
Proposed branch: | lp:~vthompson/music-app/fix-1478162 | ||||
Merge into: | lp:music-app | ||||
Diff against target: |
25 lines (+3/-2) 2 files modified
app/components/Helpers/ContentHubHelper.qml (+1/-1) debian/changelog (+2/-1) |
||||
To merge this branch: | bzr merge lp:~vthompson/music-app/fix-1478162 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Andrew Hayzen | Approve | ||
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Review via email: mp+265885@code.launchpad.net |
Commit message
* Fix to allow the LibraryEmptyState to work again
Description of the change
* Fix to allow the LibraryEmptyState to work again
A minor bug in the ContentHubHelper was causing the LibraryEmptyState to not work properly. It may have been causing other issues that we were working around. So things related to ContentHub should probably be retested for surface level failures.
I want this to land prior to releasing the next revision of the app to the app store.
To post a comment you must log in.
PASSED: Continuous integration, rev:889 91.189. 93.70:8080/ job/music- app-ci/ 1349/ 91.189. 93.70:8080/ job/music- app-vivid- amd64-ci/ 201
http://
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild: 91.189. 93.70:8080/ job/music- app-ci/ 1349/rebuild
http://