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

Updated TimeLineSettings to Use since_id #9

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

Conversation

dpparekh
Copy link

Per our discussion, I updated the timeline settings to take since_id. This will allow me to provide a minimum/since "tweet" id that I want to get data from.

The suggestion you had made to update searchsettings wouldn't have worked since per your comment earlier Twitter API doesn't allow one to search based on user for that you have to use timeline and timeline doesn't provide a way to get Tweets based on date, you have to use since_id.

Look it over and let me know what you think.

Dhaval

I updated the code to allow to pull timeline from a specific unqiue
tweet id.
Updated timelinesettings to use since_id only if it is available.
@andyhutch77
Copy link
Owner

Thanks, I'll add this to the next set of changes released. I think I can do it in a way that is more configurable but will keep your existing changes. 👍 Thanks.

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.

2 participants