Thread toot notification mails by conversation (#5061)
* Thread toot notification mails by conversation * Make codeclimate happy and avoid potential mis-threading
This commit is contained in:
		| @ -8,6 +8,7 @@ class NotificationMailer < ApplicationMailer | ||||
|     @status = notification.target_status | ||||
|  | ||||
|     locale_for_account(@me) do | ||||
|       thread_by_conversation(@status.conversation) | ||||
|       mail to: @me.user.email, subject: I18n.t('notification_mailer.mention.subject', name: @status.account.acct) | ||||
|     end | ||||
|   end | ||||
| @ -27,6 +28,7 @@ class NotificationMailer < ApplicationMailer | ||||
|     @status  = notification.target_status | ||||
|  | ||||
|     locale_for_account(@me) do | ||||
|       thread_by_conversation(@status.conversation) | ||||
|       mail to: @me.user.email, subject: I18n.t('notification_mailer.favourite.subject', name: @account.acct) | ||||
|     end | ||||
|   end | ||||
| @ -37,6 +39,7 @@ class NotificationMailer < ApplicationMailer | ||||
|     @status  = notification.target_status | ||||
|  | ||||
|     locale_for_account(@me) do | ||||
|       thread_by_conversation(@status.conversation) | ||||
|       mail to: @me.user.email, subject: I18n.t('notification_mailer.reblog.subject', name: @account.acct) | ||||
|     end | ||||
|   end | ||||
| @ -67,4 +70,13 @@ class NotificationMailer < ApplicationMailer | ||||
|            ) | ||||
|     end | ||||
|   end | ||||
|  | ||||
|   private | ||||
|  | ||||
|   def thread_by_conversation(conversation) | ||||
|     return if conversation.nil? | ||||
|     msg_id = "<conversation-#{conversation.id}.#{conversation.created_at.strftime('%Y-%m-%d')}@#{Rails.configuration.x.local_domain}>" | ||||
|     headers['In-Reply-To'] = msg_id | ||||
|     headers['References'] = msg_id | ||||
|   end | ||||
| end | ||||
|  | ||||
		Reference in New Issue
	
	Block a user