Skip to content

Conversation

@ggazzo
Copy link
Member

@ggazzo ggazzo commented Sep 13, 2022

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@ggazzo ggazzo force-pushed the feat/apply-composet branch from 745e859 to ae28c58 Compare September 14, 2022 03:22
@codecov
Copy link

codecov bot commented Sep 14, 2022

Codecov Report

Merging #26864 (5ad0b2e) into develop (6d3b20d) will increase coverage by 0.18%.
The diff coverage is 73.07%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #26864      +/-   ##
===========================================
+ Coverage    40.86%   41.04%   +0.18%     
===========================================
  Files          790      790              
  Lines        17903    17835      -68     
  Branches      1927     1929       +2     
===========================================
+ Hits          7316     7321       +5     
+ Misses       10295    10220      -75     
- Partials       292      294       +2     
Flag Coverage Δ
e2e 41.04% <73.07%> (+0.18%) ⬆️

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

@ggazzo ggazzo changed the base branch from develop to new/ui-composer September 14, 2022 15:52
@ggazzo ggazzo changed the base branch from new/ui-composer to develop September 14, 2022 15:52
@ggazzo ggazzo changed the base branch from develop to new/ui-composer September 14, 2022 19:50
@ggazzo ggazzo changed the base branch from new/ui-composer to develop September 15, 2022 04:26
@ggazzo ggazzo force-pushed the feat/apply-composet branch from e17ad36 to cea4f76 Compare September 15, 2022 04:59
@ggazzo ggazzo changed the title Chore: Rewrite Message composer in react Chore: Move Omnichannel Composer to react Sep 15, 2022
@ggazzo ggazzo marked this pull request as ready for review September 15, 2022 16:30
@ggazzo ggazzo requested review from a team as code owners September 15, 2022 16:30
@RocketChat RocketChat deleted a comment from lgtm-com bot Sep 15, 2022
@MartinSchoeler MartinSchoeler self-requested a review September 15, 2022 17:36
Copy link
Member

@MartinSchoeler MartinSchoeler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I try to open a chat via current-chats page this happens, both when trying to join (using the button) or when I open I chat I'm part of, refreshing fixes this

Screen Shot 2022-09-15 at 2 59 09 PM

Screen Shot 2022-09-15 at 3 00 00 PM

Copy link
Member

@MartinSchoeler MartinSchoeler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When trying to remove a chat from "hold" this error happens

Screen Shot 2022-09-15 at 3 09 25 PM

Screen Shot 2022-09-15 at 3 07 05 PM

@MartinSchoeler MartinSchoeler dismissed their stale review September 15, 2022 21:34

Both of the issues are happening on develop, out of the context of this PR

MartinSchoeler
MartinSchoeler previously approved these changes Sep 15, 2022
@ggazzo ggazzo added stat: QA skipped stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Sep 15, 2022
@ggazzo ggazzo changed the title Chore: Move Omnichannel Composer to react Chore: Move Omnichannel Room Footer to react Sep 15, 2022
@kodiakhq kodiakhq bot merged commit 34c07d2 into develop Sep 15, 2022
@kodiakhq kodiakhq bot deleted the feat/apply-composet branch September 15, 2022 23:17
@tassoevan tassoevan mentioned this pull request Oct 13, 2022
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.

4 participants