Skip to content

Conversation

@gabriellsh
Copy link
Member

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

In the sidebar, choose extended mode.
I'll use the channel GENERAL as the example.

With user one, join GENERAL.
With another user, join GENERAL.
On the first user, click the homepage button, or open another room, so that the room GENERAL is NOT open.

With user 2, send a message to channel GENERAL.
Now, send a different message.

On user 1, in the sidebar, the preview message should be the last sent message, which is not the case.

Further comments

@gabriellsh gabriellsh requested a review from a team as a code owner November 14, 2022 15:28
@gabriellsh gabriellsh added this to the 5.4.0 milestone Nov 14, 2022
@codecov
Copy link

codecov bot commented Nov 14, 2022

Codecov Report

Merging #27259 (1e4bfef) into develop (1c5f950) will increase coverage by 0.24%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #27259      +/-   ##
===========================================
+ Coverage    40.89%   41.14%   +0.24%     
===========================================
  Files          854      819      -35     
  Lines        18751    17929     -822     
  Branches      2053     1986      -67     
===========================================
- Hits          7669     7377     -292     
+ Misses       10791    10267     -524     
+ Partials       291      285       -6     
Flag Coverage Δ
e2e 41.14% <100.00%> (+0.24%) ⬆️

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

@ggazzo ggazzo merged commit ca29977 into develop Nov 14, 2022
@ggazzo ggazzo deleted the fix/extendedPreview branch November 14, 2022 18:59
@ggazzo ggazzo modified the milestones: 5.4.0, 5.3.2 Nov 18, 2022
@ggazzo ggazzo mentioned this pull request Nov 18, 2022
@ggazzo ggazzo mentioned this pull request Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants