Merge lp:~mardy/account-plugins/lp1429911 into lp:~online-accounts/account-plugins/rtm-14.09

Proposed by David Barth on 2015-03-24
Status: Merged
Approved by: Alberto Mardegan on 2015-03-25
Approved revision: 145
Merged at revision: 144
Proposed branch: lp:~mardy/account-plugins/lp1429911
Merge into: lp:~online-accounts/account-plugins/rtm-14.09
Diff against target: 14 lines (+2/-1)
1 file modified
data/providers/facebook.provider.in.in (+2/-1)
To merge this branch: bzr merge lp:~mardy/account-plugins/lp1429911
Reviewer Review Type Date Requested Status
David Barth (community) Approve on 2015-03-24
Review via email: mp+253977@code.launchpad.net

Commit message

Don't use v1.0 permissions
Add ResponseType to Facebook plugin

Description of the change

Don't use v1.0 permissions
Add ResponseType to Facebook plugin

To post a comment you must log in.
David Barth (dbarth) wrote :

Don't use v1.0 permissions
Add ResponseType to Facebook plugin

David Barth (dbarth) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/providers/facebook.provider.in.in'
2--- data/providers/facebook.provider.in.in 2014-09-24 08:27:39 +0000
3+++ data/providers/facebook.provider.in.in 2015-03-24 16:23:15 +0000
4@@ -16,8 +16,9 @@
5 <setting name="Host">www.facebook.com</setting>
6 <setting name="AuthPath">/dialog/oauth</setting>
7 <setting name="RedirectUri">https://www.facebook.com/connect/login_success.html</setting>
8+ <setting name="ResponseType">token</setting>
9 <setting name="Display">popup</setting>
10- <setting name="Scope" type="as">['publish_stream','read_stream','status_update','user_photos','friends_photos','xmpp_login']</setting>
11+ <setting name="Scope" type="as">['publish_actions','read_stream','user_photos']</setting>
12 <setting name="ClientId">@FACEBOOK_CLIENT_ID@</setting>
13 <setting name="AllowedSchemes" type="as">['https','http']</setting>
14 </group>

Subscribers

People subscribed via source and target branches