roschaefer · mz99 · Jul 28, 2018 · Jul 28, 2018 · Jul 29, 2018 · Jul 29, 2018
diff --git a/backend/app/controllers/application_controller.rb b/backend/app/controllers/application_controller.rb
@@ -6,6 +6,7 @@ class ApplicationController < ActionController::API
   before_action :set_paper_trail_whodunnit
   before_action :set_locale
   before_action :set_raven_context
+  before_action :set_last_login
 
   def set_locale
     I18n.locale = user_locale || guest_locale
@@ -25,6 +26,12 @@ def user_locale
     current_user&.locale
   end
 
+  def set_last_login
+    if current_user
+      current_user.update_columns(last_login: Time.current)
+    end
+  end
+
   def set_raven_context
     Raven.user_context(id: current_user.id) if current_user
     Raven.extra_context(params: params.to_unsafe_h, url: request.url)

diff --git a/backend/app/mailers/user_mailer.rb b/backend/app/mailers/user_mailer.rb
@@ -3,4 +3,9 @@ def auth0_migration(user)
     @user = user
     mail(to: @user.email, subject: 'Auth0 Migration')
   end
+
+  def monthly_news(user)
+    @user = user
+    mail(to: @user.email, subject: 'New happenings at Rundfunk Mitbestimmen')
+  end
 end
diff --git a/backend/app/models/user.rb b/backend/app/models/user.rb
@@ -63,4 +63,14 @@ def update_and_reverse_geocode(params)
     end
     save
   end
+
+  def reasons_for_notifications
+    reasons = []
+    broadcasts_created_since_last_login = Broadcast.where('created_at >= ?', last_login)
+    reasons << :recently_created_broadcasts if broadcasts_created_since_last_login.count >= 5
+    distributed_sum = impressions.to_a.sum { |i| i[:amount].to_f }
+    reasons << :no_given_amount_for_supported_broadcasts if impressions.to_a.any? { |r| r[:response] == 'positive' } && distributed_sum.zero?
+    reasons << :unbalanced_distribution if impressions.to_a.any? { |r| r[:response] == 'positive' && r[:amount] == 17.5 }
+    reasons
+  end
 end
diff --git a/backend/app/views/user_mailer/monthly_news.html.erb b/backend/app/views/user_mailer/monthly_news.html.erb
@@ -0,0 +1,38 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta content='text/html; charset=UTF-8' http-equiv='Content-Type' />
+  </head>
+<body>
+  <h2>Hello <%= @user.name %>, </h2>
+
+  <p>Welcome to another month's happenings with the new stuff this month: </p>
+
+  <% if self.reasons_for_notifications.include? :recently_created_broadcasts  %>
+    <p> Here are the new broadcasts at Rundfunk Mitbestimmen since you last logged in:
+      <% broadcasts_created_since_last_login.each do |b| %>
+        <ul><%= b.title %></ul>
+      <% end %>
+    </p>
+  <% end %>
+
+  <% if self.reasons_for_notifications.include? :no_given_amount_for_supported_broadcasts  %>
+    <p> It looks like you have supported broadcasts without actually distributing your funds for your
+      selected broadcasts. Please log in and distribute funds for your favorite programs!
+    </p>
+  <% end %>
+
+  <% if self.reasons_for_notifications.include? :unbalanced_distribution  %>
+    <p> It looks like you have distributed all of your funds to one particular broadcast, are you sure
+      that was your intention? If not, then please log in and explore more broadcasts and distribute your funds
+      to those. If that was your intention, then please ignore this message!
+    </p>
+  <% end %>
+
+  <p>Thank you for being a valued member of Rundfunk Mitbestimmen! We hope you will check back soon to see
+  these new updates! </p>
+
+  <p>Best Regards, <br>
+  The Rundfunk Mitbestimmen team </p>
+</body>
+</html>
diff --git a/backend/app/views/user_mailer/monthly_news.text.erb b/backend/app/views/user_mailer/monthly_news.text.erb
@@ -0,0 +1,28 @@
+Hello <%= @user.name %>,
+
+Welcome to another month's happenings with the new stuff this month:
+
+<% if self.reasons_for_notifications.include? :recently_created_broadcasts  %>
+   Here are the new broadcasts at Rundfunk Mitbestimmen since you last logged in:
+    <% broadcasts_created_since_last_login.each do |b| %>
+      <%= b.title %>
+    <% end %>
+<% end %>
+
+
+<% if self.reasons_for_notifications.include? :no_given_amount_for_supported_broadcasts  %>
+  It looks like you have supported broadcasts without actually distributing your funds for your
+  selected broadcasts. Please log in and distribute funds for your favorite programs!
+<% end %>
+
+<% if self.reasons_for_notifications.include? :unbalanced_distribution  %>
+  It looks like you have distributed all of your funds to one particular broadcast, are you sure
+  that was your intention? If not, then please log in and explore more broadcasts and distribute your funds
+  to those. If that was your intention, then please ignore this message!
+<% end %>
+
+Thank you for being a valued member of Rundfunk Mitbestimmen! We hope you will check back soon to see
+these new updates!
+
+Best Regards,
+The Rundfunk Mitbestimmen team
diff --git a/backend/db/migrate/20180719134105_add_last_login_to_user.rb b/backend/db/migrate/20180719134105_add_last_login_to_user.rb
@@ -0,0 +1,5 @@
+class AddLastLoginToUser < ActiveRecord::Migration[5.1]
+  def change
+    add_column :users, :last_login, :datetime
+  end
+end
diff --git a/backend/lib/tasks/monthly_mailer.rake b/backend/lib/tasks/monthly_mailer.rake
@@ -0,0 +1,9 @@
+namespace :monthly_mailer do
+  desc "Send mail alerting non-active users new happenings on Rundfunk Mitbestimmen"
+
+  task monthly_news: :environment do
+    User.find_each do |user|
+      UserMailer.monthly_news(user).deliver_now
+    end
+  end
+end
diff --git a/backend/spec/models/user_spec.rb b/backend/spec/models/user_spec.rb
@@ -8,6 +8,35 @@
   let(:liked_broadcast) { create(:impression, response: :positive, user: user).broadcast }
   let(:unsupported_broadcast) { create(:impression, response: :neutral, user: user).broadcast }
 
+  describe '#reasons_for_notifications' do
+    subject { user.reasons_for_notifications }
+
+    context 'by default' do
+      before do
+        user.update_columns(last_login: Time.current)
+      end
+      it { is_expected.to be_empty }
+    end
+
+    context 'many new broadcasts since last login' do
+      before do
+        user.update_columns(last_login: 2.months.ago)
+        create_list(:broadcast, 20)
+      end
+      it { is_expected.to include(:recently_created_broadcasts) }
+    end
+
+    context 'forgot to distribute money for supported broadcasts' do
+      before { create_list(:impression, 3, user: user, response: :positive, amount: nil) }
+      it { is_expected.to include(:no_given_amount_for_supported_broadcasts) }
+    end
+
+    context 'quite an unbalanced distribution, was it on purpose?' do
+      before { create(:impression, user: user, response: :positive, amount: 17.5) }
+      it { is_expected.to include(:unbalanced_distribution) }
+    end
+  end
+
   describe 'update_and_reverse_geocode' do
     subject { user.update_and_reverse_geocode(params) }
     let(:user) { create(:user) }