Skip to content

Instantly share code, notes, and snippets.

@bogdan
Created April 21, 2015 16:27
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save bogdan/7f499d05feb2a7dc62f1 to your computer and use it in GitHub Desktop.
Save bogdan/7f499d05feb2a7dc62f1 to your computer and use it in GitHub Desktop.
zz.diff
commit 5ac232d175d11e9126e973e3fa2f75f66b6bc318
Author: Bogdan Gusiev <agresso@gmail.com>
Date: Tue Apr 21 17:44:24 2015 +0300
Don't push yaml to sidekiq
diff --git a/app/controllers/concerns/public_template_rendering.rb b/app/controllers/concerns/public_template_rendering.rb
index 5c158e2..cd2178f 100644
--- a/app/controllers/concerns/public_template_rendering.rb
+++ b/app/controllers/concerns/public_template_rendering.rb
@@ -47,12 +47,12 @@ module PublicTemplateRendering
render_page_view_setup("offers_show", offer, params[:responsive])
else
unless current_visitor.bot?
- ActivitiesWorker.perform_async(:create!, YAML::dump(
- offer: @offer,
- visitor: current_visitor,
+ ActivitiesWorker.perform_async(
+ offer_id: @offer.id,
+ visitor_id: current_visitor.id,
action: "affiliate_signup_page",
- traffic_source: @offer.origin.traffic_source,
- ))
+ traffic_source_id: @offer.origin.traffic_source_id,
+ )
end
render_page_view_setup("affiliate_members_new", offer, params[:responsive])
end
diff --git a/app/models/email_offer_share.rb b/app/models/email_offer_share.rb
index a13060d..192deb3 100644
--- a/app/models/email_offer_share.rb
+++ b/app/models/email_offer_share.rb
@@ -245,12 +245,12 @@ class EmailOfferShare < OfferShare
yield # deliver_template
save!
- ActivitiesWorker.perform_async(:create!, YAML::dump(
- offer: offer,
+ ActivitiesWorker.perform_async(
+ offer_id: offer_id,
action: 'share',
- sharing_channel: site.sharing_channel_for(:reminder),
- visitor: offer.visitor,
- ))
+ sharing_channel_id: site.sharing_channel_for(:reminder).id,
+ visitor_id: offer.visitor.id,
+ )
end
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment