diff --git a/app/jobs/send_template_message_job.rb b/app/jobs/send_template_message_job.rb index f88b30494..125ed3e0b 100644 --- a/app/jobs/send_template_message_job.rb +++ b/app/jobs/send_template_message_job.rb @@ -38,7 +38,7 @@ class SendTemplateMessageJob < ApplicationJob return unless operator.present? && issue.present? # receivers = receivers.where.not(id: operator&.id) receivers_string, content, notification_url = MessageTemplate::IssueAtme.get_message_content(receivers, operator, issue) - Notice::Write::CreateService.call(receivers_string, content, notification_url, source, {operator_id: operator.id, issue_id: issue.id}, 2) + Notice::Write::CreateService.call(receivers_string, content, notification_url, source, {operator_id: operator.id, issue_id: issue.id}, 2, operator_id) when 'IssueChanged' operator_id, issue_id, change_params = args[0], args[1], args[2] operator = User.find_by_id(operator_id) @@ -236,7 +236,7 @@ class SendTemplateMessageJob < ApplicationJob return unless operator.present? && pull_request.present? # receivers = receivers.where.not(id: operator&.id) receivers_string, content, notification_url = MessageTemplate::PullRequestAtme.get_message_content(receivers, operator, pull_request) - Notice::Write::CreateService.call(receivers_string, content, notification_url, source, {operator_id: operator.id, pull_request_id: pull_request.id}, 2) + Notice::Write::CreateService.call(receivers_string, content, notification_url, source, {operator_id: operator.id, pull_request_id: pull_request.id}, 2, operator_id) when 'PullRequestChanged' operator_id, pull_request_id, change_params = args[0], args[1], args[2] operator = User.find_by_id(operator_id) diff --git a/app/views/users/messages/_message.json.jbuilder b/app/views/users/messages/_message.json.jbuilder index 9f6f0c504..075613ce9 100644 --- a/app/views/users/messages/_message.json.jbuilder +++ b/app/views/users/messages/_message.json.jbuilder @@ -14,11 +14,20 @@ when 1 when 2 json.type "atme" json.sender do - sender = User.find_by_id(message["sender"]) - if sender.present? - json.partial! '/users/user_simple', locals: {user: sender} - else - json.nil + sender = $redis_cache.hgetall("v2-owner-common:#{message["sender"]}") + if sender.blank? + sender = User.find_by_id(message["sender"]) || User.find_by_id(JSON.parse(message['extra'])['operator_id']) + if sender.present? + json.partial! '/users/user_simple', locals: {user: sender} + else + json.nil + end + else + json.id message["sender"] + json.type sender['type'] + json.name sender['name'] + json.login sender['login'] + json.image_url sender['avatar_url'] end end end