-
Notifications
You must be signed in to change notification settings - Fork 32
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
Destroys expired messages daily #51
Conversation
Testing 🏁 @trevormast Can you rebase and push again to trigger a new Travis build? Thanks! |
a0d0a3e
to
ef5c3d8
Compare
Done! 🐐 |
@@ -18,6 +18,8 @@ gem 'slim-rails' | |||
gem 'therubyracer' | |||
gem 'thin' | |||
gem 'whenever' | |||
gem 'sidekiq' | |||
gem 'sinatra' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can move this to the :development
group since it's only needed for the Sidekiq dashboard which we only set routes for in the dev
env.
Adds Timecop gem
Only requires sidekiq/web in development
ef5c3d8
to
ee45329
Compare
@jgrevich good call. Made changes and rebased! |
Awesome stuff, thanks @trevormast! 🎉 My apologies for taking so long to get to this. I'm just beginning to pickup work on this project and will be more responsive going forward. |
Addresses #42
Needs configuration for redis in production.
Adds specs to cover each.