Merge pull request #197 from just3ws/master

just3ws · just3ws · commit 02ed896fdcd7 · 2014-09-04T07:48:00.000-05:00
In production the arguments to sidekiq are being cast to string
diff --git a/app/workers/protip_mailer_popular_protips_worker.rb b/app/workers/protip_mailer_popular_protips_worker.rb
@@ -3,6 +3,12 @@ class ProtipMailerPopularProtipsWorker
   sidekiq_options queue: :low
 
   def perform(from, to)
+
+    # In development the arguments are the correct type but in production
+    # they have to be recast from string back to date types :D
+    from = Time.zone.parse(from.to_s)
+    to = Time.zone.parse(to.to_s)
+
     protips = ProtipMailer::Queries.popular_protips(from, to)
 
     User.find_each(batch_size: 100) do |user|
