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.

Sign up for GitHub

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

Jump to bottom

Auto follower functionality #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Auto follower functionality #19

wants to merge 1 commit into from

Conversation

kousiksatish
Copy link

This PR fixes #6

@kousiksatish
Copy link
Author

There are few tasks left to implement in this function

  • Check cases of no friends, no followers
  • Check flags like user.following and user.follow_request_sent to check status of whether follow was successful
  • Check cases where follow requires approval
  • Check efficient ways to handle all errors that can occur during follow post call
  • Twitter has "400 per user; 1000 per app" limitation per user. Need to review response message in this case

@vinitshahdeo vinitshahdeo added enhancement New feature or request good first issue Good for newcomers hacktoberfest Send PR and earn a free t-shirt up-for-grabs Solve the issue and raise your first PR labels Oct 11, 2019
@vinitshahdeo
Copy link
Owner

@kousiksatish Thanks for the PR. I'll review the code soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest Send PR and earn a free t-shirt up-for-grabs Solve the issue and raise your first PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Auto follower [Follow for Follow]
2 participants
@kousiksatish @vinitshahdeo