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

Update README.md file, use DB as storage #1607

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

Conversation

shqear93
Copy link
Contributor

@alexreisner I would like to hear your input about this, this is very optional addtition to the reademe file that might be helpful for some users

@shqear93 shqear93 force-pushed the feature/update-readme-use-db-as-storage branch from 59a6af0 to 692ab36 Compare February 26, 2023 21:34
@alexreisner
Copy link
Owner

@shqear93 I like this. Two suggestions:

  • This should go in the "Caching" section, right above the last line, which is in italics ("Before you implement caching...").
  • I'm trying to reduce the amount of text in the README. Can this be a single line?: "In addition to conventional cache stores like Redis, it's possible to keep your cache in the database using ActiveRecord. For example see this gist."

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.

None yet

2 participants