Merge lp:~mardy/ubuntu-system-settings-online-accounts/fix-auth-with-params into lp:ubuntu-system-settings-online-accounts

Proposed by Alberto Mardegan
Status: Merged
Approved by: Ken VanDine
Approved revision: 69
Merged at revision: 69
Proposed branch: lp:~mardy/ubuntu-system-settings-online-accounts/fix-auth-with-params
Merge into: lp:ubuntu-system-settings-online-accounts
Diff against target: 12 lines (+1/-1)
1 file modified
src/module/OAuth.qml (+1/-1)
To merge this branch: bzr merge lp:~mardy/ubuntu-system-settings-online-accounts/fix-auth-with-params
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Ken VanDine Approve
Review via email: mp+191405@code.launchpad.net

Commit message

Fix OAuth authentication with parameters

Fix an obvious JS error.

Description of the change

Fix OAuth authentication with parameters

Fix an obvious JS error.

To post a comment you must log in.
Revision history for this message
Ken VanDine (ken-vandine) wrote :

Obvious fix

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/module/OAuth.qml'
--- src/module/OAuth.qml 2013-10-09 21:56:30 +0000
+++ src/module/OAuth.qml 2013-10-16 13:28:27 +0000
@@ -136,7 +136,7 @@
136 var parameters = {136 var parameters = {
137 "X-PageComponent": "file:///usr/share/signon-ui/online-accounts-ui/Page.qml"137 "X-PageComponent": "file:///usr/share/signon-ui/online-accounts-ui/Page.qml"
138 }138 }
139 for (p in authenticationParameters) {139 for (var p in authenticationParameters) {
140 parameters[p] = authenticationParameters[p]140 parameters[p] = authenticationParameters[p]
141 }141 }
142 __isAuthenticating = true142 __isAuthenticating = true

Subscribers

People subscribed via source and target branches