LeadMagnet/leadchat
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

fix: Delete content_attributes in message destroy API (#4184)

Browse Source
This commit is contained in:
Pranav Raj S
2022-03-15 22:01:16 +05:30
committed by GitHub
parent 8c8c5a77c8
commit 4b748e2c8c
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/controllers/api/v1/accounts/conversations/messages_controller.rb
View File

@@ -13,7 +13,7 @@ class Api::V1::Accounts::Conversations::MessagesController < Api::V1::Accounts::
def destroy def destroy
ActiveRecord::Base.transaction do ActiveRecord::Base.transaction do
message.update!(content: I18n.t('conversations.messages.deleted'), deleted: true) message.update!(content: I18n.t('conversations.messages.deleted'), content_attributes: { deleted: true })
message.attachments.destroy_all message.attachments.destroy_all
end end
end end

3
spec/controllers/api/v1/accounts/conversations/messages_controller_spec.rb
View File

@@ -163,7 +163,7 @@ RSpec.describe 'Conversation Messages API', type: :request do
end end
describe 'DELETE /api/v1/accounts/{account.id}/conversations/:conversation_id/messages/:id' do describe 'DELETE /api/v1/accounts/{account.id}/conversations/:conversation_id/messages/:id' do
let(:message) { create(:message, account: account) } let(:message) { create(:message, account: account, content_attributes: { bcc_emails: ['hello@chatwoot.com'] }) }
let(:conversation) { message.conversation } let(:conversation) { message.conversation }
context 'when it is an unauthenticated user' do context 'when it is an unauthenticated user' do
@@ -188,6 +188,7 @@ RSpec.describe 'Conversation Messages API', type: :request do
expect(response).to have_http_status(:success) expect(response).to have_http_status(:success)
expect(message.reload.content).to eq 'This message was deleted' expect(message.reload.content).to eq 'This message was deleted'
expect(message.reload.deleted).to eq true expect(message.reload.deleted).to eq true
expect(message.reload.content_attributes['bcc_emails']).to eq nil
end end
end end