[go: nahoru, domu]

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

Allow a timer to be used for follow actions to avoid breaching Twitter terms of use #43

Merged
merged 2 commits into from
Jun 16, 2015

Conversation

mgoodfellow
Copy link
Contributor

Follow actions tend to be a difficult one when it comes to the Twitter API.

Therefore, this allows a couple of extra configuration fields (FOLLOW_BACKOFF_MIN_SECONDS and FOLLOW_BACKOFF_MAX_SECONDS) to cause the bot to sleep a random interval between the two periods thereby seeming less like a bot.

Also added future functions to allow this to be run on Python < 3 interpreters

…ng Twitter terms of use

ADDED: Compatibility for Python < 3
@rhiever
Copy link
Owner
rhiever commented Jun 12, 2015

This is great; thank you for the PR! My only hesitation is making this a required feature. I think it would be better as an optional feature. Could you please take it out of the "required parameters" list and add the if statements to only send the bot to sleep if they have the sleep parameters set?

@mgoodfellow
Copy link
Contributor Author

Hi @rhiever,

Thanks for the feedback - I have removed the timers from the required parameters list, and also refactored a little bit so its a bit more intelligent and robust.

Any more comments are more than welcome!

@edworboys
Copy link

+1

rhiever pushed a commit that referenced this pull request Jun 16, 2015
Allow a timer to be used for follow actions to avoid breaching Twitter terms of use
@rhiever rhiever merged commit b5dd0da into rhiever:master Jun 16, 2015
@mgoodfellow mgoodfellow deleted the follow-backoff-timers branch June 16, 2015 12:40
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.

3 participants