Merge lp:~mardy/ubuntu-system-settings-online-accounts/show-u1-1559506 into lp:ubuntu-system-settings-online-accounts

Proposed by Alberto Mardegan
Status: Merged
Approved by: dobey
Approved revision: 345
Merged at revision: 346
Proposed branch: lp:~mardy/ubuntu-system-settings-online-accounts/show-u1-1559506
Merge into: lp:ubuntu-system-settings-online-accounts
Diff against target: 13 lines (+2/-1)
1 file modified
plugins/OnlineAccountsPlugin/application-manager.cpp (+2/-1)
To merge this branch: bzr merge lp:~mardy/ubuntu-system-settings-online-accounts/show-u1-1559506
Reviewer Review Type Date Requested Status
dobey (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+290587@code.launchpad.net

Commit message

Always show the UbuntuOne provider in the providers list

Description of the change

Always show the UbuntuOne provider in the providers list

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
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
1=== modified file 'plugins/OnlineAccountsPlugin/application-manager.cpp'
2--- plugins/OnlineAccountsPlugin/application-manager.cpp 2016-03-08 08:48:40 +0000
3+++ plugins/OnlineAccountsPlugin/application-manager.cpp 2016-03-31 10:06:15 +0000
4@@ -227,7 +227,8 @@
5 Q_FOREACH(const Accounts::Service &service, allServices) {
6 if (providers.contains(service.provider())) continue;
7
8- if (!manager->applicationList(service).isEmpty()) {
9+ if (!manager->applicationList(service).isEmpty() ||
10+ service.provider() == "ubuntuone") {
11 providers.append(service.provider());
12 }
13 }

Subscribers

People subscribed via source and target branches