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

Adding option to store cache either in disk or in transients #26

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

Conversation

viniciusbarros
Copy link

@viniciusbarros viniciusbarros commented Dec 7, 2018

As saving cache in database is not a viable option for much projects that I work with, I would like to see the plugin being able to store cache either in disk or in transients.

How it works:

  • Get response
  • serialize it and save it disk
  • when reading, check if cache file exists + check if it has not expired based on it's creation/modification time.

What was added:

  • 2 new options in dashboard (disk or transient + path to cache in disk)
  • method to delete cache in disk
  • Generating and storing cache in disk

What was changed:

  • When deleting cache we first check if the cache is transient or disk before executing

@Mte90
Copy link

Mte90 commented Mar 16, 2019

any news for this?

@airesvsg
Copy link
Owner

Hi @Mte90,

I'll rewrite to improve the code.

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.

3 participants