Merge lp:~mikemc/ubuntu-system-settings-online-accounts/fix-remove-account-button into lp:ubuntu-system-settings-online-accounts
Proposed by
Mike McCracken
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Alberto Mardegan | ||||||||
Approved revision: | 42 | ||||||||
Merged at revision: | 42 | ||||||||
Proposed branch: | lp:~mikemc/ubuntu-system-settings-online-accounts/fix-remove-account-button | ||||||||
Merge into: | lp:ubuntu-system-settings-online-accounts | ||||||||
Diff against target: |
12 lines (+1/-1) 1 file modified
src/module/Options.qml (+1/-1) |
||||||||
To merge this branch: | bzr merge lp:~mikemc/ubuntu-system-settings-online-accounts/fix-remove-account-button | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alberto Mardegan (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+183939@code.launchpad.net |
Commit message
- Fix account removal button for OAuth accounts.
Description of the change
- Fix account removal button for OAuth accounts.
The account removal button was adding an extra undefined parameter to PopupUtils.open(), which isn't needed.
To post a comment you must log in.
PASSED: Continuous integration, rev:42 jenkins. qa.ubuntu. com/job/ ubuntu- system- settings- online- accounts- ci/25/ jenkins. qa.ubuntu. com/job/ ubuntu- system- settings- online- accounts- saucy-amd64- ci/25
http://
Executed test runs:
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins: 8080/job/ ubuntu- system- settings- online- accounts- ci/25/rebuild
http://