Merge lp://staging/~marcustomlinson/unity-scope-click/lp-1378186 into lp://staging/unity-scope-click

Proposed by Marcus Tomlinson
Status: Merged
Approved by: dobey
Approved revision: 453
Merged at revision: 459
Proposed branch: lp://staging/~marcustomlinson/unity-scope-click/lp-1378186
Merge into: lp://staging/unity-scope-click
Diff against target: 133 lines (+23/-15)
5 files modified
libclickscope/click/scope_activation.cpp (+17/-6)
libclickscope/click/scope_activation.h (+1/-4)
scope/clickapps/apps-query.cpp (+3/-2)
scope/clickapps/apps-query.h (+1/-1)
scope/clickapps/apps-scope.cpp (+1/-2)
To merge this branch: bzr merge lp://staging/~marcustomlinson/unity-scope-click/lp-1378186
Reviewer Review Type Date Requested Status
dobey (community) Approve
PS Jenkins bot continuous-integration Pending
Review via email: mp+295023@code.staging.launchpad.net

Commit message

Return apps scope to root department if current department becomes empty after an uninstall

To post a comment you must log in.
Revision history for this message
Marcus Tomlinson (marcustomlinson) wrote :

I have tested this on the phone. If you'd like to confirm, here's a build: https://code.launchpad.net/~unity-api-team/+archive/ubuntu/dev-build-3

Revision history for this message
Marcus Tomlinson (marcustomlinson) wrote :

This is now in silo 47

Revision history for this message
dobey (dobey) :
review: Needs Fixing
453. By Marcus Tomlinson

uninstall_sucess_f -> uninstall_success_f

Revision history for this message
Marcus Tomlinson (marcustomlinson) :
Revision history for this message
dobey (dobey) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches