Merge lp:~ahayzen/music-app/remix-recent-playlists-freeze into lp:music-app/remix
Proposed by
Andrew Hayzen
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Victor Thompson | ||||
Approved revision: | 715 | ||||
Merged at revision: | 716 | ||||
Proposed branch: | lp:~ahayzen/music-app/remix-recent-playlists-freeze | ||||
Merge into: | lp:music-app/remix | ||||
Diff against target: |
116 lines (+54/-12) 3 files modified
MusicStart.qml (+16/-0) common/SongsPage.qml (+37/-11) meta-database.js (+1/-1) |
||||
To merge this branch: | bzr merge lp:~ahayzen/music-app/remix-recent-playlists-freeze | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ubuntu Phone Apps Jenkins Bot | continuous-integration | Approve | |
Victor Thompson | Approve | ||
Review via email: mp+239928@code.launchpad.net |
Commit message
* Fix for editing/deleting a playlist from Recent causing freeze
* Fix for rename in Recent attempting to add
Description of the change
* Fix for editing/deleting a playlist from Recent causing freeze
* Fix for rename in Recent attempting to add
To post a comment you must log in.
LGTM! The cases that previously froze the app and/or left the playlists on the recent tab when there was no playlist with that name has been fixed.
Again, we should look at removing the need for the playlist deadlock timer.