From c998c84bc090b75f282a7b0a95b11c158e65a614 Mon Sep 17 00:00:00 2001 From: Pranav Raj S Date: Fri, 24 Feb 2023 01:54:53 -0800 Subject: [PATCH] fix: Name missing when email is collected via email hook (#6530) - The name is not updated when the email is updated via the email collect message. This PR fixes that. --- app/controllers/api/v1/widget/messages_controller.rb | 2 +- spec/controllers/api/v1/widget/messages_controller_spec.rb | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/app/controllers/api/v1/widget/messages_controller.rb b/app/controllers/api/v1/widget/messages_controller.rb index 7ff77be32..6287c94ee 100644 --- a/app/controllers/api/v1/widget/messages_controller.rb +++ b/app/controllers/api/v1/widget/messages_controller.rb @@ -17,7 +17,7 @@ class Api::V1::Widget::MessagesController < Api::V1::Widget::BaseController @message.update!(submitted_email: contact_email) ContactIdentifyAction.new( contact: @contact, - params: { email: contact_email } + params: { email: contact_email, name: contact_name } ).perform else @message.update!(message_update_params[:message]) diff --git a/spec/controllers/api/v1/widget/messages_controller_spec.rb b/spec/controllers/api/v1/widget/messages_controller_spec.rb index 56485f9fc..06c0b46a9 100644 --- a/spec/controllers/api/v1/widget/messages_controller_spec.rb +++ b/spec/controllers/api/v1/widget/messages_controller_spec.rb @@ -138,6 +138,7 @@ RSpec.describe '/api/v1/widget/messages', type: :request do message.reload expect(message.submitted_email).to eq(email) expect(message.conversation.contact.email).to eq(email) + expect(message.conversation.contact.name).to eq(email.split('@')[0]) end end