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

[BUG] Message read status in active chat #16765

Closed
ankar84 opened this issue Mar 2, 2020 · 2 comments
Closed

[BUG] Message read status in active chat #16765

ankar84 opened this issue Mar 2, 2020 · 2 comments

Comments

@ankar84
Copy link

ankar84 commented Mar 2, 2020

Description:

Now we are using in test mode option Admin -> General -> Notifications -> Always notify mobile and I see some not comfortable behavior of that function in active chats.
I get mobile notification even for active chats where I'm actively writing messages

Steps to reproduce:

  1. Enable Always notify mobile in Admin -> General -> Notifications
  2. Go to any chat to talk with someone
  3. Chat window is active, cursor in message area, I'm looking on the screen and reading everything
  4. You get new message in active chat with red line (unread messages) and immediately you get push notification on mobile client

Expected behavior:

Mobile notification should send only for totally unread messages.
Messages in active chat (RC.Electron or Browser on focus, on top, chat is opened) shouldn't be mark as unread.

Actual behavior:

In screenshot you can see cursor in message area and RC.Electron 2.17.7 is on top, is on focus, chat is open, I intently looking at chat and reading all new messages immediately.
image

Server Setup Information:

  • Version of Rocket.Chat Server: 3.0.2
  • Operating System: CentOS7
  • Deployment Method: docker
  • Number of Running Instances: 12
  • DB Replicaset Oplog: Enabled
  • NodeJS Version: v12.14.0
  • MongoDB Version: 4.0.13

Client Setup Information

  • Desktop App or Browser Version: 2.17.7 and Chrome 80.0.3987.122
  • Operating System: Windows 10 1809

Additional context

I understand, that if user walked away from his computer, locked his session with open active chat and then came back, it is conveniently to see all unread messages. But you can add a condition if user status was changed - chat not active anymore and unread messages may appear on all new messages.

@aleadco
Copy link

aleadco commented Mar 2, 2020

This error was also reported in RocketChat/Rocket.Chat.ReactNative#1794

@ankar84
Copy link
Author

ankar84 commented Apr 23, 2020

Closed with #17357

@ankar84 ankar84 closed this as completed Apr 23, 2020
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

No branches or pull requests

2 participants