Online Accounts: Account plugins

Merge lp://staging/~mardy/account-plugins/use-new-api into lp://staging/account-plugins

  1. use-new-api
  2. Merge into trunk
Proposed by Alberto Mardegan
Status: Merged
Approved by: Víctor R. Ruiz
Approved revision: 54
Merged at revision: 56
Proposed branch: lp://staging/~mardy/account-plugins/use-new-api
Merge into: lp://staging/account-plugins
Diff against target: 16 lines (+1/-3)
1 file modified
tools/account-console (+1/-3)
To merge this branch: bzr merge lp://staging/~mardy/account-plugins/use-new-api
Reviewer Review Type Date Requested Status
Víctor R. Ruiz (community) Approve
jenkins (community) continuous-integration Needs Fixing
Alvaro Lopez Ortega Pending
David King Pending
Review via email: mp+116198@code.staging.launchpad.net

Description of the change

Use new API to get credentials ID

This fixes a crash, actually :-)

To post a comment you must log in.
Revision history for this message
jenkins (martin-mrazik+qa) wrote :

FAILED: Continuous integration, rev:54
http://s-jenkins:8080/job/online-accounts-account-plugins-ci/1/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/online-accounts-account-plugins-ci/./label=precise-clean-i386/1/console

review: Needs Fixing (continuous-integration)
Revision history for this message
Víctor R. Ruiz (vrruiz) wrote :

Looks fine.

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