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

Add useCache option for standard useAxios call #34

Merged

simoneb merged 1 commit into simoneb:master from verekia:master

Oct 12, 2019

Contributor

verekia commented Sep 29, 2019 •

edited

Fixes #33

I tried but I couldn't manage to write a working test for it.

Something like spying on lru-cache get would be nice, but I'm not sure how to spy on it, since it's instantiated in index.js with new LRU().


          Add useCache option for standard useAxios call

87b498d

verekia changed the title ~~Add useCache option for standard useAxios call. Fix #33~~ Add useCache option for standard useAxios call

simoneb mentioned this pull request

useCache option with tests #37

Merged

simoneb merged commit 87b498d into simoneb:master

fabn mentioned this pull request

Option to disable cache with makeUseAxios #199

Closed

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