Merge lp:~andrewsomething/account-plugins/instagram into lp:account-plugins
Proposed by
Andrew Starr-Bochicchio
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | VĂctor R. Ruiz | ||||
Approved revision: | 102 | ||||
Merged at revision: | 101 | ||||
Proposed branch: | lp:~andrewsomething/account-plugins/instagram | ||||
Merge into: | lp:account-plugins | ||||
Diff against target: |
191 lines (+86/-1) 10 files modified
Makefile.am (+3/-0) configure.ac (+15/-0) data/providers/instagram.provider.in.in (+27/-0) data/services/instagram-microblog.service.in (+21/-0) data/webkit-options/api.instagram.com.conf (+2/-0) debian/account-plugin-instagram.install (+3/-0) debian/changelog (+3/-0) debian/control (+8/-0) debian/rules (+3/-1) po/POTFILES.skip (+1/-0) |
||||
To merge this branch: | bzr merge lp:~andrewsomething/account-plugins/instagram | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ken VanDine | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Alberto Mardegan (community) | Approve | ||
Andrew Starr-Bochicchio (community) | Approve | ||
Review via email: mp+158442@code.launchpad.net |
Commit message
Added account-
Description of the change
Add support for instagram as first step to adding instagram to friends-app.
To post a comment you must log in.
Hi Andrew, , thank you very much for your contribution. We'd love to see it incorporate in to Ubuntu, but first I need to ask you to sign the contributor agreement. It's a straight forward process, and you can find full details here:
http:// www.canonical. com/contributor s
Please let me know once you've completed the form.
On the code front, I'd like to suggest you a change: >['likes+ comments' ]</setting>
87 + <setting type="as" name="Scope"
Instead of "likes+comments", try to write this as: >['likes' , 'comments' ]</setting>
<setting type="as" name="Scope"
Our oauth plugin should concatenate the two strings as appropriate, but just to be on the safe side please test it once more, after having made the change.