[Bug 1638769] Re: Need more information from account manager

2016-11-21 Thread Michi Henning
I've opened a separate bug here, seeing this one is closed: Bug #1643732 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1638769 Title: Need more information from account manager To manage notificat

[Bug 1638769] Re: Need more information from account manager

2016-11-21 Thread Michi Henning
Hmmm... This is not working for me. I have a Google Drive account configured. When I call GetAccounts(), the account is returned. But the account information I get doesn't include what we need. We need the translated provider name (e.g. "Google Drive"), the icon name, and the DBus name (e.g. "com.

[Bug 1638769] Re: Need more information from account manager

2016-11-17 Thread Alberto Mardegan
I checked the error with the Account Tester app: the error message is misleading, the reason for the failure is that probably you didn't install the click, so the click hook that should create the .application file didn't run, and therefore the service doesn't know who this app is. -- You receive

[Bug 1638769] Re: Need more information from account manager

2016-11-16 Thread Alberto Mardegan
I'll investigate why are you are getting that error with the tester app; it works fine on the phone. Anyway, yes, the output you are getting from gdbus is expected, because you are not specifying the applicationId in your call to GetAccounts. You need to ship an .application file along with your a

[Bug 1638769] Re: Need more information from account manager

2016-11-16 Thread Alberto Mardegan
** Changed in: webapps-sprint Status: In Progress => Fix Released -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1638769 Title: Need more information from account manager To manage notificati

[Bug 1638769] Re: Need more information from account manager

2016-11-16 Thread James Henstridge
I tried downloading your accounts-tester repo, and ran "qmlscene provider-tester/app.qml". It shows an empty window with a "create a new account" button at the bottom. When I click that, it shows an empty "Choose provider" page. On the console, it printed the following: virtual void OnlineA

[Bug 1638769] Re: Need more information from account manager

2016-11-15 Thread Alberto Mardegan
At the moment, we don't properly handle the case when new providers are installed on the system: the client will continue seeing the old provider's list. You can file a bug for that, but I don't think this is a critical issue. The client retrieve the provider list (along with the existing accounts

[Bug 1638769] Re: Need more information from account manager

2016-11-15 Thread James Henstridge
Is this actually working as intended? Looking at the changes in the latest package it only affects the GetAccounts() call, so we don't get back the additional account metadata from the RequestAccess() method, or the AccountChanged signal. Further more, the extra "services" return value from GetAc

[Bug 1638769] Re: Need more information from account manager

2016-11-15 Thread Launchpad Bug Tracker
This bug was fixed in the package online-accounts-api - 0.1+17.04.20161110-0ubuntu1 --- online-accounts-api (0.1+17.04.20161110-0ubuntu1) zesty; urgency=medium * Add provider information to public API (LP: #1627001, #1638769) -- Alberto Mardegan Thu, 10 Nov 2016 09:13:34 +

[Bug 1638769] Re: Need more information from account manager

2016-11-04 Thread Alberto Mardegan
** Also affects: webapps-sprint Importance: Undecided Status: New ** Changed in: webapps-sprint Status: New => In Progress ** Changed in: webapps-sprint Assignee: (unassigned) => Alberto Mardegan (mardy) ** Changed in: webapps-sprint Importance: Undecided => High ** Cha

[Bug 1638769] Re: Need more information from account manager

2016-11-04 Thread Alberto Mardegan
** Branch linked: lp:~mardy/online-accounts-api/providers-1627001 -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1638769 Title: Need more information from account manager To manage notifications abo

[Bug 1638769] Re: Need more information from account manager

2016-11-03 Thread Michi Henning
Thanks Mardy! Passing icon data over DBus would be sub-optimal, yes. For clicks, I don't think we'd have a problem with the app resolving the icon name. (I assume that the default apparmor profile for apps will allow this.) I don't know what the story will be for snaps. Presumably, there will be s

[Bug 1638769] Re: Need more information from account manager

2016-11-03 Thread Alberto Mardegan
I agree that this information is needed by the clients. The question is about the icon: the client might be unable (due to apparmor confinement) to read the provider icon, especially if this is coming from another click or snap package. If, as I understand it - please confirm this - the storage

[Bug 1638769] Re: Need more information from account manager

2016-11-03 Thread Alberto Mardegan
** Changed in: online-accounts-api (Ubuntu) Status: New => Confirmed ** Changed in: online-accounts-api (Ubuntu) Assignee: (unassigned) => Alberto Mardegan (mardy) -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bu