Sutty/trabajo-afectivo
5
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fixes #2275 - Display name not used

Browse source
This commit is contained in:
Romit Choudhary 2021-10-28 11:33:02 +02:00
parent 827c18b471
commit 23c5f43fca
2 changed files with 17 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/assets/javascripts/app/lib/app_post/utils.coffee
View file

@ -1327,7 +1327,7 @@ class App.Utils
if type is 'email' && !e.attrs.value.match(/@/) || e.attrs.value.match(/\s/)
e.preventDefault()
return false
e.attrs.label = e.attrs.value
e.attrs.label ||= e.attrs.value
true
)
App.Delay.set(a, 500, undefined, 'tags')

19
spec/system/ticket/update/email_reply_spec.rb
View file

@ -17,7 +17,7 @@ RSpec.describe 'Ticket > Update > Email Reply', current_user_id: -> { current_us
it 'shows error dialog when updated value is an invalid email' do
within(:active_content) do
click_reply
click_email_reply
find('.token').double_click
find('.js-to', visible: false).sibling('.token-input').set('test')
@ -30,7 +30,7 @@ RSpec.describe 'Ticket > Update > Email Reply', current_user_id: -> { current_us
it 'updates article when updated value is a valid email' do
within(:active_content) do
click_reply
click_email_reply
find('.token').double_click
find('.js-to', visible: false).sibling('.token-input').set('user@test.com')
@ -43,7 +43,20 @@ RSpec.describe 'Ticket > Update > Email Reply', current_user_id: -> { current_us
end
def click_reply
context 'when a new recipient is added in email reply' do
it 'shows both name and email in token' do
click_email_reply
find('.js-to', visible: false).sibling('.token-input').set(customer.email)
find('ul.ui-autocomplete li:first-child', visible: false).click
within all('.token-label')[1] do
expect(page).to have_text("#{customer.firstname} #{customer.lastname} <#{customer.email}>")
end
end
end
def click_email_reply
click '.js-ArticleAction[data-type=emailReply]'
end