Merge pull request #155 from zach-chai/email_subject

add recording name to subject

app/mailers/notification_mailer.rb

@@ -21,9 +21,12 @@ class NotificationMailer < ActionMailer::Base
     @user = user
     @recording = rec
     @room_url = meeting_room_url(resource: 'rooms', id: user.encrypted_id)
+
+    # Need this because URL doesn't respect the relative_url_root by default
     unless @room_url.include? "#{Rails.configuration.relative_url_root}/"
       @room_url = @room_url.split('/rooms/').join("#{Rails.configuration.relative_url_root}/rooms/")
     end
-    mail(to: user.email, subject: t('.subject'))
+
+    mail(to: user.email, subject: t('.subject', recording: @recording[:name]))
   end
 end

config/locales/en-us.yml

@@ -119,7 +119,7 @@ en-US:
       phrase3_html: "This recording is currently <b>unlisted</b>, which means only users with the above direct link can view the recording."
       phrase4: "To make this recording public for everyone, visit"
       phrase5: "and change its visibility from unlisted to public.  After the recording is public, anyone with the above URL can view it."
-      subject: "Your recording is ready!"
+      subject: "Your recording is ready - %{recording}"
       table:
         ended: Ended
         name: Name