Merge pull request #375 from doubleunion/scholarship-email-improvement

Add user and text to scholarship email

app/mailers/dues_mailer.rb

@@ -13,7 +13,7 @@ def scholarship_requested(user, reason)
     @reason = reason
 
     mail(
-      to: [SCHOLARSHIP_EMAIL],
+      to: [SCHOLARSHIP_EMAIL, @user.email],
       subject: "New DU Scholarship Request",
     )
   end

app/views/dues_mailer/scholarship_requested.html.haml

@@ -10,6 +10,9 @@
 %p
   Please go to the app and approve or deny this person's scholarship, then email them to let them know. Please also notify the membership coordinator ([email protected]) that the person is on the scholarship list, so they can make sure the person's scholarship status is reflected in the membership management system (such as setting them up with Google Drive and Calendar if they're a brand-new member).
 
+%p
+  Scholarship requests should be responded to within two weeks.
+
 %p
   Yours Truly,
   %br/

spec/controllers/members/dues_controller_spec.rb

@@ -190,7 +190,7 @@
 
       it "sends an email" do
         expect { subject }.to change { ActionMailer::Base.deliveries.count }.by(1)
-        expect(ActionMailer::Base.deliveries.last.to).to eq(["[email protected]"])
+        expect(ActionMailer::Base.deliveries.last.to).to eq(["[email protected]", member.email])
         expect(ActionMailer::Base.deliveries.last.body).to include "Lemurs are pretty great"
       end
     end