-
Notifications
You must be signed in to change notification settings - Fork 594
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
Comments
I think it would be cleaner as part of the existing node - but have not thought in detail about the implications of that. |
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 The app lets you retweet with comment - but that doesn't appear to be part of the @baruchiro you are most welcome to explore this and propose something along these lines. |
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: 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. |
was this ever updated to support retweets? I tried using |
Hi, the
node-red-node-twitter
is not supporting retweets, since it uses/statuses/update.json
, and the retweet is done bystatuses/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.
The text was updated successfully, but these errors were encountered: