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

Chore: remove request button when chat is close #27768

Merged
merged 2 commits into from
Jan 24, 2023

Conversation

filipemarins
Copy link
Contributor

Proposed changes (including videos or screenshots)

Remove the request button when the chat is already closed.

Issue(s)

Steps to test or reproduce

Further comments

Request email transcript modal.

Before:
Screenshot 2023-01-16 at 17 26 16

After:
Screenshot 2023-01-16 at 17 27 46

@codecov
Copy link

codecov bot commented Jan 16, 2023

Codecov Report

Merging #27768 (5b3ee65) into feat/chat-transcript (85f61de) will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@                   Coverage Diff                    @@
##           feat/chat-transcript   #27768      +/-   ##
========================================================
- Coverage                 42.93%   42.89%   -0.05%     
========================================================
  Files                       818      818              
  Lines                     17267    17267              
  Branches                   1944     1944              
========================================================
- Hits                       7414     7407       -7     
- Misses                     9585     9597      +12     
+ Partials                    268      263       -5     
Flag Coverage Δ
e2e 42.89% <ø> (-0.05%) ⬇️

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

@filipemarins filipemarins marked this pull request as ready for review January 17, 2023 12:44
@filipemarins filipemarins requested review from a team as code owners January 17, 2023 12:44
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Jan 24, 2023
@filipemarins filipemarins merged commit d296755 into feat/chat-transcript Jan 24, 2023
@filipemarins filipemarins deleted the oc-490 branch January 24, 2023 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants