-
Notifications
You must be signed in to change notification settings - Fork 11.3k
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] [Livechat] Unread messages badge #26843
Conversation
5169ce0
to
0784c2f
Compare
Codecov Report
@@ Coverage Diff @@
## develop #26843 +/- ##
===========================================
- Coverage 40.95% 40.81% -0.14%
===========================================
Files 791 791
Lines 17809 17809
Branches 1928 1928
===========================================
- Hits 7293 7269 -24
- Misses 10222 10245 +23
- Partials 294 295 +1
Flags with carried forward coverage won't be shown. Click here to find out more. |
…livechat-reload-messages-unreaded
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add e2e tests to prevent this issue from happening again?
…livechat-reload-messages-unreaded
…livechat-reload-messages-unreaded
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some grammar suggestions on the tests nomenclature
Co-authored-by: Martin Schoeler <martin.schoeler@rocket.chat>
Co-authored-by: Martin Schoeler <martin.schoeler@rocket.chat>
Co-authored-by: Martin Schoeler <martin.schoeler@rocket.chat>
Co-authored-by: Martin Schoeler <martin.schoeler@rocket.chat>
Co-authored-by: Martin Schoeler <martin.schoeler@rocket.chat>
Co-authored-by: Martin Schoeler <martin.schoeler@rocket.chat>
Proposed changes (including videos or screenshots)
OC-169
Even if the page was reopening, Livechat will inform unread messages
Issue(s)
close #25529
Steps to test or reproduce
Further comments