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

Unable to get API calls count #62

Closed
Menelion opened this Issue Jan 12, 2016 · 3 comments

Comments

Projects
None yet
2 participants
@Menelion
Collaborator

Menelion commented Jan 12, 2016

In TheQube there's no way to get the rate limit. Ctrl+Win+A for some reason doesn't work.

@Menelion Menelion added the bug label Jan 12, 2016

@Menelion Menelion self-assigned this Jan 12, 2016

@Menelion Menelion added this to the 1.0 milestone Jan 12, 2016

craigbrett17 added a commit to craigbrett17/TheQube that referenced this issue Jul 16, 2016

Fix issue #62 API call counts for twitter
The Twython library seems to suggest that now it no longer has a rate_limit function
(or whatever was being used), and now these things are retrieved using
get_lastfunction_header, with the appropriate header to retrieve the value.
Possibly due to upstream Twitter API changes?
@craigbrett17

This comment has been minimized.

Show comment
Hide comment
@craigbrett17

craigbrett17 Jul 23, 2016

Contributor

@Oire what's the procedure for me getting my fix in for things like this? Do I raise it as a pull request or do you pull in things from other people's repos? I'm not sure what the project's way of working is on that level.

Contributor

craigbrett17 commented Jul 23, 2016

@Oire what's the procedure for me getting my fix in for things like this? Do I raise it as a pull request or do you pull in things from other people's repos? I'm not sure what the project's way of working is on that level.

@Menelion

This comment has been minimized.

Show comment
Hide comment
@Menelion

Menelion Jul 23, 2016

Collaborator

@craigbrett17, just do a pull request, and I'll review and merge it.

Collaborator

Menelion commented Jul 23, 2016

@craigbrett17, just do a pull request, and I'll review and merge it.

Menelion pushed a commit that referenced this issue Aug 10, 2016

Andre Polykanine A.K.A. Menelion Elensúlë
Merge pull request #66 from craigbrett17/62-api-calls-count
Fix issue #62 API call counts for twitter
@Menelion

This comment has been minimized.

Show comment
Hide comment
@Menelion

Menelion Aug 10, 2016

Collaborator

Closing via #66.

Collaborator

Menelion commented Aug 10, 2016

Closing via #66.

@Menelion Menelion closed this Aug 10, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment