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

Reply method #55

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Reply method #55

wants to merge 3 commits into from

Conversation

Yutsa
Copy link

@Yutsa Yutsa commented Jul 10, 2015

Here's the pull resquest for my reply method.

Count is the number of people to tweet to.
"""

result = self.search_tweets(keyword, 2)
Copy link
Owner

Choose a reason for hiding this comment

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

Please edit this line to take the count parameter instead of 2.

@rhiever
Copy link
Owner

rhiever commented Jul 20, 2015

This looks great. Thank you for the PR. I added a comment on one line that needs to be fixed before I can merge it.

I think it would also be helpful to keep a list of the tweets that have been responded to, so this reply method doesn't reply to the same tweets again and again. Thoughts?

@Yutsa
Copy link
Author

Yutsa commented Jul 20, 2015

Yeah I updated the method since I made the PR. It stores the id of the tweet it already replied in a file now and some more things.
The thing is I modified the code for my personal use so maybe there are some things that are not really needed.
I tried to keep only the necessary code.
I'll update the PR, just tell me if I have to modify anything.

@alfons3992
Copy link

hey man, i don't understand one thing. i put the source on the bottom of the original twitterfollowbot and created a file called "messages.txt" i put in that file: "hello, lol, ..." etc. but my bot doesn't reply to any tweets. also put a word in the "...." = keyword bracket and count=5. what am i missing?

@music-phobia
Copy link

Could anyone make a function to automatically DM to those who started following you?

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 this pull request may close these issues.

4 participants