diff --git a/app/services/message_templates/hook_execution_service.rb b/app/services/message_templates/hook_execution_service.rb index 4307cf2c9..1cac42a16 100644 --- a/app/services/message_templates/hook_execution_service.rb +++ b/app/services/message_templates/hook_execution_service.rb @@ -2,13 +2,18 @@ class MessageTemplates::HookExecutionService pattr_initialize [:message!] def perform - return if conversation.campaign.present? - return if conversation.last_incoming_message.blank? + if conversation.campaign.present? + Rails.logger.debug { "[OutOfOffice][#{conversation.id}] Not triggering templates because conversation has a campaign" } + return + end + if conversation.last_incoming_message.blank? + Rails.logger.debug { "[OutOfOffice][#{conversation.id}] Not triggering templates because there is no incoming message" } + return + end + + Rails.logger.info "[OutOfOffice][#{conversation.id}] Triggering templates for conversation ##{conversation.id}" trigger_templates - rescue StandardError => e - Rails.logger.error "[OutOfOffice][#{conversation.id}] Error triggering templates: #{e.message}" - raise e end private @@ -25,16 +30,37 @@ class MessageTemplates::HookExecutionService def should_send_out_of_office_message? # should not send if its a tweet message - return false if conversation.tweet? + if conversation.tweet? + Rails.logger.debug { "[OutOfOffice][#{conversation.id}] Not sending out-of-office message because it's a tweet conversation" } + return false + end # should not send for outbound messages - return false unless message.incoming? + unless message.incoming? + Rails.logger.debug { "[OutOfOffice][#{conversation.id}] Not sending out-of-office message because the message is outgoing" } + return false + end # prevents sending out-of-office message if an agent has sent a message in last 5 minutes # ensures better UX by not interrupting active conversations at the end of business hours - return false if conversation.messages.outgoing.where(private: false).exists?(['created_at > ?', 5.minutes.ago]) + if conversation.messages.outgoing.where(private: false).exists?(['created_at > ?', 5.minutes.ago]) + Rails.logger.debug { "[OutOfOffice][#{conversation.id}] Not sending out-of-office message because an agent responded in the last 5 minutes" } + return false + end - inbox.out_of_office? && conversation.messages.today.template.empty? && inbox.out_of_office_message.present? + can_send = inbox.out_of_office? && conversation.messages.today.template.empty? && inbox.out_of_office_message.present? + + if can_send + Rails.logger.info "[OutOfOffice][#{conversation.id}] Sending out-of-office message for conversation ##{conversation.id}" + else + reasons = [] + reasons << 'inbox not in out-of-office mode' unless inbox.out_of_office? + reasons << 'conversation already has a template message today' unless conversation.messages.today.template.empty? + reasons << 'inbox has no out-of-office message configured' unless inbox.out_of_office_message.present? + Rails.logger.debug { "[OutOfOffice][#{conversation.id}] Not sending out-of-office message because: #{reasons.join(', ')}" } + end + + can_send rescue StandardError => e Rails.logger.error("[OutOfOffice][#{conversation.id}] Error triggering out of office message: #{e.message}") ChatwootExceptionTracker.new(e, account: conversation.account).capture_exception