Multiple Notifications for the same message

Open
#3,301 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
42/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Quiet
Tech stack
electron, typescript
Domain
desktop

Research direction

Start by reproducing the issue on Windows 10 with Rocket.Chat Desktop 4.14.0 and Server 7.13, following the three reported steps. Trace the desktop notification handling after the app loses focus and verify that each incoming message produces only one notification; the issue is done when repeated notifications no longer occur.

Written by the indexing model from the issue text.

Description

type: bug
Search before asking
  • I had searched in the issues and found no similar issues.
Operating System
  • macOS
  • Windows
  • Linux
Operating System Version

Windows 10

It happens on the web browser too?

No, it just happens on the Desktop app

Rocket.Chat Desktop App Version

4.14.0

Rocket.Chat Server Version

7.13

Describe the bug

Since the recent update any message notification is sent multiple times. Around 5-7 times per message.
Restarting Rocket.Chat does not seem to fix it.

Image
How to Reproduce
  1. Un-focus Rocket.Chat
  2. Get message
  3. Multiple notifications will show
Describe your Expected behavior

Only one notification per message

Anything else

No response

Are you willing to submit a code contribution?
  • Yes, I am willing to submit a Pull Request!
Dominant language
TypeScript
Stars
1.7k
Forks
835
Avg merge
1d 21h
Merged PRs (30d)
15

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from RocketChat/Rocket.Chat.Electron

All issues in RocketChat/Rocket.Chat.Electron

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.