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

Node-red-node-twitter retweet support #834

Open
baruchiro opened this issue Sep 6, 2021 · 4 comments · May be fixed by #835
Open

Node-red-node-twitter retweet support #834

baruchiro opened this issue Sep 6, 2021 · 4 comments · May be fixed by #835

Comments

@baruchiro
Copy link

Hi, the node-red-node-twitter is not supporting retweets, since it uses /statuses/update.json, and the retweet is done by statuses/retweet/:id.

I want to create a PR to add this support. I think since it is a different interface- it needs only the tweet id to retweet it, I will create a new node for it.

Or maybe the preferred way is to implement it inside the Twitter Out node?

WDYT?

I asked in the forum, but no answers there.

@dceejay
Copy link
Member

dceejay commented Sep 6, 2021

I think it would be cleaner as part of the existing node - but have not thought in detail about the implications of that.

@knolleary
Copy link
Member

Agreed it would be better to have this in the existing output node.

There are lots of features of the Twitter API we don't support today, so it would be good to keep that in mind when adding individual features.

For this, I'd suggest if the received message includes msg.retweet then use its value as the tweet id to retweet.

The app lets you retweet with comment - but that doesn't appear to be part of the /status/retweet/ endpoint. It would make sense if msg.payload is set to some text as well as msg.retweet set for that to be a retweet with comment - its just a matter of figuring out the right API to use for that.

@baruchiro you are most welcome to explore this and propose something along these lines.

@baruchiro
Copy link
Author

OK, about retweet with text, I'm not sure I will take this. Maybe if it will be on my way.

About the existing output node. First, I agree. But the reason I was thinking to implement a new output node is in order to support the simplest flow, input to output:

image

Today, if you are wiring such flow, it will tweet the input tweet, as text (or any other content) but not like a real retweet supported by Twitter.

With implementing in the existing node, this simple flow will require another node to take the tweetID and put it in the payload.

baruchiro added a commit to baruchiro/node-red-nodes that referenced this issue Sep 10, 2021
@baruchiro baruchiro linked a pull request Sep 10, 2021 that will close this issue
6 tasks
baruchiro added a commit to baruchiro/node-red-nodes that referenced this issue Sep 23, 2021
baruchiro added a commit to baruchiro/node-red-nodes that referenced this issue Sep 23, 2021
@moshed
Copy link

moshed commented Aug 14, 2022

was this ever updated to support retweets? I tried using msg.retweet with no success

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

Successfully merging a pull request may close this issue.

4 participants