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

[feature] Allow configuring caching #49

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

Conversation

hallelujah
Copy link

As with Rails, this allows to configure the cache store through config.rails_heroicon_cache_store

By default it is set to :memory_store.

Closes #48

As with Rails, this allows to configure the cache store
through `config.rails_heroicon_cache_store`

By default it is set to :memory_store.

Closes abeidahmed#48
So that all tests benefits from Railtie initialization
@abeidahmed
Copy link
Owner

Thanks for the contribution, @hallelujah. I can see the benefits of using a cache store, but I'll need to benchmark the SVG render speed when compared to the existing hash store.

FYI: This is the issue #34 that hash store was trying to fix.

@hallelujah
Copy link
Author

Thanks for the answer. Let's see if we do not lose too much in performance using MemoryStore

Also for my case, I would like to be able to share the cache between multiple processes in production

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.

[feature] Allow customization of the cache through ActiveSupport caching
2 participants