From ce400a36d7b12413e500918fcaea4c74b1d937d1 Mon Sep 17 00:00:00 2001 From: Shivam Mishra Date: Thu, 30 Oct 2025 13:36:39 +0530 Subject: [PATCH] feat: Always process email content (#12734) Co-authored-by: Muhsin Keloth --- app/builders/messages/message_builder.rb | 2 +- spec/builders/messages/message_builder_spec.rb | 13 ------------- 2 files changed, 1 insertion(+), 14 deletions(-) diff --git a/app/builders/messages/message_builder.rb b/app/builders/messages/message_builder.rb index 12a74ed9c..857d901e5 100644 --- a/app/builders/messages/message_builder.rb +++ b/app/builders/messages/message_builder.rb @@ -23,7 +23,7 @@ class Messages::MessageBuilder process_emails # When the message has no quoted content, it will just be rendered as a regular message # The frontend is equipped to handle this case - process_email_content if @account.feature_enabled?(:quoted_email_reply) + process_email_content @message.save! @message end diff --git a/spec/builders/messages/message_builder_spec.rb b/spec/builders/messages/message_builder_spec.rb index 2eb4dbf90..64a1aad62 100644 --- a/spec/builders/messages/message_builder_spec.rb +++ b/spec/builders/messages/message_builder_spec.rb @@ -199,19 +199,6 @@ describe Messages::MessageBuilder do expect(message.content_attributes.dig('email', 'text_content', 'reply')).to eq 'Regular message content' end - it 'does not process custom email content when quoted_email_reply feature is disabled' do - account.disable_features('quoted_email_reply') - params = ActionController::Parameters.new({ - content: 'Regular message content', - email_html_content: '

Custom HTML content

' - }) - - message = described_class.new(user, conversation, params).perform - - expect(message.content_attributes.dig('email', 'html_content')).to be_nil - expect(message.content_attributes.dig('email', 'text_content')).to be_nil - end - it 'does not process custom email content for private messages' do params = ActionController::Parameters.new({ content: 'Regular message content',