Skip to content
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

Get image from DM #463

Closed
wants to merge 1 commit into from
Closed

Get image from DM #463

wants to merge 1 commit into from

Conversation

ericrkl
Copy link

@ericrkl ericrkl commented Apr 20, 2016

Add function getimg to download immage attached to direct messages.

Modified function http.

Add function getimg to download immage attached to direct messages.

Modified function http.
@abraham
Copy link
Owner

abraham commented May 14, 2016

I'm a little unsure what the purpose of this is. What Twitter API method is this for?

@ericrkl
Copy link
Author

ericrkl commented May 14, 2016

It's not for a Twitter method per se, it's just to have a way to get and decode images sent in a direct message. I think is handy to "have it all" in one lib.

From twittercommunity.com
"media shared in direct messages requires authorization to view. This authorization can be presented via an authenticated twitter.com session or by signing a request with the user's access token using OAuth 1.0A."

@aliriza0
Copy link

hello.twitter trend topic system update?.my system is no work.please help me

@abraham
Copy link
Owner

abraham commented May 21, 2016

@aliriza0 if you are having problems please open a new issue.

@abraham
Copy link
Owner

abraham commented May 21, 2016

@ericrkl I'm thinking about if this is the best approach, if it would make more sense to have a method that returns a signed URL for the developer to download as they choose, or some other approach.

@ericrkl
Copy link
Author

ericrkl commented May 21, 2016

As long as there’s a way to get the image from the URL in the end.

I made a social media customer care system for clients to interact with their audience (like Hoot Suite or Sprout), and it’s common for that audience to send some photo(s) as proof of some kind, via direct message (proof of purchase, or payment, or a photo from a damaged luggage to an airline, etc). These photos need to be downloaded and kept as a record of a conversation with some person. So it’s a very useful and commonly used operation.


Eric R Kelly
Whatsapp +1 (702) 879-7944

On 21/05/2016, at 12:23 p.m., Abraham Williams notifications@github.com wrote:

@ericrkl https://github.com/ericrkl I'm thinking about if this is the best approach, if it would make more sense to have a method that returns a signed URL for the developer to download as they choose, or some other approach.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub #463 (comment)

Copy link
Owner

@abraham abraham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the best way to get this added is to copy http to a new file method. The two methods will be relatively similar except the file method will take a full URL and return the response without JSON Decoding. This will leave http untouched and avoid all the messy if in this current version of the PR.

@ericrkl
Copy link
Author

ericrkl commented Feb 23, 2017 via email

@abraham abraham modified the milestone: v1.0 Feb 23, 2017
@abraham abraham modified the milestones: v1.0, v2.0 Sep 28, 2017
@ericrkl
Copy link
Author

ericrkl commented Oct 26, 2017 via email

@abraham abraham removed this from the v2.0 milestone Jul 3, 2018
@abraham
Copy link
Owner

abraham commented Jul 4, 2018

Related to #684 and #685.

@joelcuevas
Copy link

joelcuevas commented Feb 9, 2019

Does the "file" method proposal is still the preferred solution? I need to download DM files, if this is still valid, I could submit a PR in the next days.

@9tN4tGi72j1z4Qe

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants