diff --git a/app/mailers/digest_mailer.rb b/app/mailers/digest_mailer.rb index c992be1aa..08c1c9bf5 100644 --- a/app/mailers/digest_mailer.rb +++ b/app/mailers/digest_mailer.rb @@ -7,7 +7,6 @@ class DigestMailer < ApplicationMailer @billboards = params[:billboards] @unsubscribe = generate_unsubscribe_token(@user.id, :email_digest_periodic) - @article_design_variant = field_test(:digest_articles_design_05_02, participant: @user) subject = generate_title # set sendgrid category in the header using smtp api diff --git a/app/services/email_digest_article_collector.rb b/app/services/email_digest_article_collector.rb index 73ea46acd..75b8ee586 100644 --- a/app/services/email_digest_article_collector.rb +++ b/app/services/email_digest_article_collector.rb @@ -12,7 +12,25 @@ class EmailDigestArticleCollector def articles_to_send # rubocop:disable Metrics/BlockLength - order = Arel.sql("((score * (feed_success_score + 0.1)) - clickbait_score) DESC") + order_variant = field_test(:digest_article_ordering_05_09, participant: @user) + order = case order_variant + when "base" + Arel.sql("((score * (feed_success_score + 0.1)) - clickbait_score) DESC") + when "more_weight_on_feed_success" + Arel.sql("((score * (feed_success_score + 0.2)) - clickbait_score) DESC") + when "much_more_weight_on_feed_success" + Arel.sql("((score * (feed_success_score + 0.3)) - clickbait_score) DESC") + when "more_weight_on_clickbait" + Arel.sql("((score * (feed_success_score + 0.1)) - (clickbait_score * 0.5)) DESC") + when "much_more_weight_on_clickbait" + Arel.sql("((score * (feed_success_score + 0.1)) - (clickbait_score * 0.8)) DESC") + when "more_weight_on_comments" + Arel.sql("((score * (feed_success_score + 0.1)) - clickbait_score + comment_score) DESC") + when "much_more_weight_on_comments" + Arel.sql("((score * (feed_success_score + 0.1)) - clickbait_score + (comment_score * 2)) DESC") + else + Arel.sql("((score * (feed_success_score + 0.1)) - clickbait_score) DESC") + end instrument ARTICLES_TO_SEND, tags: { user_id: @user.id } do return [] unless should_receive_email? diff --git a/app/views/mailers/digest_mailer/digest_email.html.erb b/app/views/mailers/digest_mailer/digest_email.html.erb index b50aa9ac5..fc381f862 100644 --- a/app/views/mailers/digest_mailer/digest_email.html.erb +++ b/app/views/mailers/digest_mailer/digest_email.html.erb @@ -34,67 +34,13 @@ <% @articles.each_with_index do |article, i| %>
<%= truncate(article.description, length: 180) %>
- <% when "base_with_user" %> - <%= article.title.strip %> - <% if article.cached_user.present? %> -
-
- <%= article.cached_user.name %>
-
- <%= truncate(article.description, length: 180) %> -
- <% when "base_with_tags" %> - <%= article.title.strip %> -- <% article.decorate.cached_tag_list_array.each do |tag| %> - #<%= tag %> - <% end %> -
-- <%= truncate(article.description, length: 180) %> -
- <% when "border" %> -- <%= truncate(article.description, length: 180) %> -
-
-
- <%= article.cached_user.name %>
-
- <%= truncate(article.description, length: 180) %> -
-- <% article.decorate.cached_tag_list_array.each do |tag| %> - #<%= tag %> - <% end %> -
-- <%= truncate(article.description, length: 180) %> -
-