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 GetAccounts() seems
to be empty here:

    $ gdbus call --session --dest com.ubuntu.OnlineAccounts.Manager 
--object-path /com/ubuntu/OnlineAccounts/Manager --method 
com.ubuntu.OnlineAccounts.Manager.GetAccounts {}
    ([(uint32 8, {'authMethod': <3>, 'displayName': 
<'james@localhost:8888/owncloud'>, 'serviceId': <'storage-provider-owncloud'>, 
'settings/host': <'http://localhost:8888/owncloud'>}), ...], @aa{sv} [])

Was there any reason not to just ship this extra account information as
part of the existing a{sv} dict used by all the existing methods?

-- 
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 about this bug go to:
https://bugs.launchpad.net/webapps-sprint/+bug/1638769/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to