Merge lp:~jpds/account-plugins/linkedin-expanded-access into lp:account-plugins

Proposed by Jonathan Davies on 2013-03-21
Status: Merged
Approved by: Alberto Mardegan on 2013-03-22
Approved revision: 97
Merged at revision: 97
Proposed branch: lp:~jpds/account-plugins/linkedin-expanded-access
Merge into: lp:account-plugins
Diff against target: 13 lines (+2/-1)
1 file modified
data/providers/linkedin.provider.in.in (+2/-1)
To merge this branch: bzr merge lp:~jpds/account-plugins/linkedin-expanded-access
Reviewer Review Type Date Requested Status
Alberto Mardegan (community) 2013-03-21 Approve on 2013-03-22
PS Jenkins bot (community) continuous-integration Approve on 2013-03-21
Review via email: mp+154809@code.launchpad.net

Commit message

Expand requested access for LinkedIn plugin so Gwibber/Friends can access network updates/messages/contacts.

Description of the change

Expand our requested access for LinkedIn so Friends/Gwibber can access more pieces.

To post a comment you must log in.
Alberto Mardegan (mardy) wrote :

Looks fine!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/providers/linkedin.provider.in.in'
2--- data/providers/linkedin.provider.in.in 2013-03-21 15:38:30 +0000
3+++ data/providers/linkedin.provider.in.in 2013-03-21 21:15:35 +0000
4@@ -19,7 +19,8 @@
5 <setting name="ClientSecret">@LINKEDIN_CONSUMER_SECRET@</setting>
6 <setting name="RedirectUri">https://www.linkedin.com/</setting>
7 <setting name="ResponseType">code&amp;state=sajfhaskncask</setting>
8- <setting type="as" name="Scope">['r_fullprofile','r_emailaddress']</setting>
9+ <setting type="as" name="Scope">['r_fullprofile',
10+ 'r_emailaddress', 'r_network', 'w_messages', 'rw_nus']</setting>
11 </group>
12 </group>
13 </group>

Subscribers

People subscribed via source and target branches

to all changes: