Merge lp:~mardy/account-plugins/lp1441476 into lp:account-plugins

Proposed by Alberto Mardegan on 2015-04-10
Status: Merged
Approved by: David Barth on 2015-04-15
Approved revision: 146
Merged at revision: 146
Proposed branch: lp:~mardy/account-plugins/lp1441476
Merge into: lp:account-plugins
Diff against target: 12 lines (+1/-1)
1 file modified
qml/facebook/Main.qml (+1/-1)
To merge this branch: bzr merge lp:~mardy/account-plugins/lp1441476
Reviewer Review Type Date Requested Status
David Barth (community) 2015-04-10 Approve on 2015-04-15
PS Jenkins bot (community) continuous-integration Approve on 2015-04-10
Review via email: mp+255777@code.launchpad.net

Commit message

The "username" field has been removed in the 2.0 API

https://developers.facebook.com/docs/apps/upgrading/

Description of the change

The "username" field has been removed in the 2.0 API

https://developers.facebook.com/docs/apps/upgrading/

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'qml/facebook/Main.qml'
2--- qml/facebook/Main.qml 2013-06-07 11:56:25 +0000
3+++ qml/facebook/Main.qml 2015-04-10 05:41:23 +0000
4@@ -13,7 +13,7 @@
5 console.log("ok")
6 console.log("response text: " + http.responseText)
7 var response = JSON.parse(http.responseText)
8- account.updateDisplayName(response.username)
9+ account.updateDisplayName(response.name)
10 globalAccountService.updateSettings({
11 'id': response.id
12 })

Subscribers

People subscribed via source and target branches

to all changes: