Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Pluggable fetch strategies #24

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

Conversation

woodhull
Copy link

@woodhull woodhull commented May 2, 2017

This PR adds the ability to have different pluggable strategies for what to do when the rate limit is exceeded.

For now there are new strategies that sleep or schedule jobs to run in the future. Added these in order to deal with issues described here #2

@coveralls
Copy link

coveralls commented May 2, 2017

Coverage Status

Coverage decreased (-3.6%) to 96.364% when pulling e308e54 on controlshift:pluggable-fetch-strategies into bcb524e on enova:master.

@coveralls
Copy link

coveralls commented May 3, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling ea6912b on controlshift:pluggable-fetch-strategies into bcb524e on enova:master.

@coveralls
Copy link

coveralls commented May 3, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 210e629 on controlshift:pluggable-fetch-strategies into bcb524e on enova:master.

@bf4
Copy link

bf4 commented Nov 20, 2019

@packrat386 PR is stale. Close or review and rebase?

@woodhull woodhull mentioned this pull request Oct 7, 2020
@rnubel
Copy link
Contributor

rnubel commented Mar 11, 2023

This PR is older than both my children combined, but are you still interested in having it merged? I have no objection to it, conceptually.

@lavaturtle
Copy link
Contributor

I'd be interested in seeing this merged. It would allow a codebase I work on to switch back to the published version of the gem instead of this fork.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants