Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] Transcript email is not sent if chat is closed by visitor #27976

Merged
merged 2 commits into from
Feb 10, 2023

Conversation

murtaza98
Copy link
Contributor

@murtaza98 murtaza98 commented Feb 8, 2023

Proposed changes (including videos or screenshots)

Issue(s)

OC-645

Steps to test or reproduce

Further comments

@KevLehman KevLehman added this to the 6.0.0 milestone Feb 8, 2023
@murtaza98 murtaza98 marked this pull request as ready for review February 8, 2023 15:44
@murtaza98 murtaza98 requested a review from a team as a code owner February 8, 2023 15:44
@codecov
Copy link

codecov bot commented Feb 9, 2023

Codecov Report

Merging #27976 (608ee47) into feat/chat-transcript (553b3a8) will increase coverage by 1.00%.
The diff coverage is n/a.

Impacted file tree graph

@@                   Coverage Diff                    @@
##           feat/chat-transcript   #27976      +/-   ##
========================================================
+ Coverage                 42.09%   43.10%   +1.00%     
========================================================
  Files                       848      821      -27     
  Lines                     17560    17043     -517     
  Branches                   2078     2003      -75     
========================================================
- Hits                       7392     7346      -46     
+ Misses                     9890     9419     -471     
  Partials                    278      278              
Flag Coverage Δ
e2e 43.10% <ø> (+1.00%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@KevLehman KevLehman merged commit 6a4c5ed into feat/chat-transcript Feb 10, 2023
@KevLehman KevLehman deleted the visitor-email-pref branch February 10, 2023 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants