feat: Adds API for retry messages in conversation (#8518)
This commit is contained in:
@@ -18,6 +18,15 @@ class Api::V1::Accounts::Conversations::MessagesController < Api::V1::Accounts::
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def retry
|
||||||
|
return if message.blank?
|
||||||
|
|
||||||
|
message.update!(status: :sent, content_attributes: {})
|
||||||
|
::SendReplyJob.perform_later(message.id)
|
||||||
|
rescue StandardError => e
|
||||||
|
render_could_not_create_error(e.message)
|
||||||
|
end
|
||||||
|
|
||||||
def translate
|
def translate
|
||||||
return head :ok if already_translated_content_available?
|
return head :ok if already_translated_content_available?
|
||||||
|
|
||||||
|
|||||||
@@ -0,0 +1 @@
|
|||||||
|
json.partial! 'api/v1/models/message', message: @message
|
||||||
@@ -84,6 +84,7 @@ Rails.application.routes.draw do
|
|||||||
resources :messages, only: [:index, :create, :destroy] do
|
resources :messages, only: [:index, :create, :destroy] do
|
||||||
member do
|
member do
|
||||||
post :translate
|
post :translate
|
||||||
|
post :retry
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
resources :assignments, only: [:create]
|
resources :assignments, only: [:create]
|
||||||
|
|||||||
@@ -234,4 +234,49 @@ RSpec.describe 'Conversation Messages API', type: :request do
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
describe 'POST /api/v1/accounts/{account.id}/conversations/:conversation_id/messages/:id/retry' do
|
||||||
|
let(:message) { create(:message, account: account, status: :failed, content_attributes: { external_error: 'error' }) }
|
||||||
|
|
||||||
|
context 'when it is an unauthenticated user' do
|
||||||
|
it 'returns unauthorized' do
|
||||||
|
post "/api/v1/accounts/#{account.id}/conversations/#{message.conversation.display_id}/messages/#{message.id}/retry"
|
||||||
|
expect(response).to have_http_status(:unauthorized)
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
context 'when it is an authenticated user with access to conversation' do
|
||||||
|
let(:agent) { create(:user, account: account, role: :agent) }
|
||||||
|
|
||||||
|
before do
|
||||||
|
create(:inbox_member, inbox: message.conversation.inbox, user: agent)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'retries the message' do
|
||||||
|
post "/api/v1/accounts/#{account.id}/conversations/#{message.conversation.display_id}/messages/#{message.id}/retry",
|
||||||
|
headers: agent.create_new_auth_token,
|
||||||
|
as: :json
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:success)
|
||||||
|
expect(message.reload.status).to eq('sent')
|
||||||
|
expect(message.reload.content_attributes['external_error']).to be_nil
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
context 'when the message id is invalid' do
|
||||||
|
let(:agent) { create(:user, account: account, role: :agent) }
|
||||||
|
|
||||||
|
before do
|
||||||
|
create(:inbox_member, inbox: message.conversation.inbox, user: agent)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'returns not found error' do
|
||||||
|
post "/api/v1/accounts/#{account.id}/conversations/#{message.conversation.display_id}/messages/99999/retry",
|
||||||
|
headers: agent.create_new_auth_token,
|
||||||
|
as: :json
|
||||||
|
|
||||||
|
expect(response).to have_http_status(:unprocessable_entity)
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
Reference in New Issue
Block a user