Merge lp:~vthompson/music-app/fix-1401730 into lp:music-app/remix
Proposed by
Victor Thompson
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Andrew Hayzen | ||||
Approved revision: | 763 | ||||
Merged at revision: | 764 | ||||
Proposed branch: | lp:~vthompson/music-app/fix-1401730 | ||||
Merge into: | lp:music-app/remix | ||||
Diff against target: |
12 lines (+1/-1) 1 file modified
music-app.qml (+1/-1) |
||||
To merge this branch: | bzr merge lp:~vthompson/music-app/fix-1401730 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Andrew Hayzen | Approve | ||
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Review via email:
|
Commit message
Use the proper model to determine if the library is loaded.
Description of the change
Use the proper model to determine if the library is loaded. This was broken when we (I) wrapped the model in a SortFilterModel to get case insensitive sorting.
To post a comment you must log in.
PASSED: Continuous integration, rev:763 91.189. 93.70:8080/ job/music- app-ci/ 1174/ 91.189. 93.70:8080/ job/generic- mediumtests- vivid/476 91.189. 93.70:8080/ job/generic- mediumtests- vivid/476/ artifact/ work/output/ *zip*/output. zip 91.189. 93.70:8080/ job/music- app-vivid- amd64-ci/ 26
http://
Executed test runs:
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild: 91.189. 93.70:8080/ job/music- app-ci/ 1174/rebuild
http://