-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
publish: support POSSE like to Instagram #326
Comments
Please also document support for this feature on https://www.brid.gy/about#like :) |
tentatively bumping to now. i don't plan to jump on it myself in the near future, but theoretically we should only need to implement the instagram API part, and flip a handful of switches elsewhere, and the rest should Just Work. |
can you fill me in on the background here -- https://github.com/snarfed/python-instagram is a fork of but it seems the latter two have diverged non-trivially. If I want a change from the Instagram/python-instagram fork, should I just export it as a diff and apply it manually? I could also merge in the upstream (Instagram/python-instagram) fork. |
thanks again for tackling this! ugh, yeah, the libs. you're right, nontrivial skew btw all three. manual patch is probably the best route...but honestly i'd love to drop the lib and just use the REST API directly. that's obviously scope creep for this, but i'm totally open to it! either one sgtm. |
cool, I agree it would be nice to pull out the lib and I'm happy to tackle that. |
Question: When a token is revoked, Instagram replies with status code 400 and error_type=OAuthAccessTokenException. This is important because later on Bridgy gets the 401 and disables the source. Should I do basically the same thing in interpret_http_exception now that it's a regular urllib2.HTTPError instead of the special InstagramAPIError? i.e. check for 'OAuthAccessTokenException' and 'APIRequiresAuthenticationError' in the body of the error message and give 401 as the error code? Thanks! |
yup, good call, that's exactly what we'd want to do. thanks! |
yay, launched! example: https://snarfed.org/2015-01-08_photo-by-thejohnnysmith-2 @kylewm, i'll let you do the honor of closing this. :P |
if I post a "like" of an Instagram photo (with a u-like-of link to an Instagram photo permalink), it would be great if I could call Bridgy Publish to POSSE that like to my account on Instagram.
The text was updated successfully, but these errors were encountered: