Skip to content
This repository has been archived by the owner on Jun 7, 2020. It is now read-only.

[BUG] Multiple copies of message is being sent when one is expected #2008

Closed
lorek123 opened this issue Jan 26, 2019 · 7 comments · Fixed by #2096
Closed

[BUG] Multiple copies of message is being sent when one is expected #2008

lorek123 opened this issue Jan 26, 2019 · 7 comments · Fixed by #2096
Assignees
Labels
Milestone

Comments

@lorek123
Copy link

Describe the bug

When user sends a message and hits multiple times send button, app will send last message multiple times. Users do this when application is not responding and not expecting to actually send it multiple times.

To Reproduce

  1. Type a message
  2. Smash the hell of the send button
  3. See the multiple occurences of the last message

Expected behavior

Only one occurence of message sent to server.

Logs

Nothing in logs.

Screenshots

image

Devices and Versions

Your Rocket.Chat.Android version: 3.2.0
Your Rocket.Chat Server version: 0.70.4
Mobile device model and OS version: Xiaomi Mi 5, LineageOS 15.1

@Pitstopper
Copy link
Contributor

I confirm this problem.

@bkraul
Copy link

bkraul commented Feb 21, 2019

Same issue here. I would suppose it is not necessarily rate limiting what is needed but to disable the send button while the message is sending. Because what it seems happening is that the message is not being delivered causing the user to tap, and then you see 6-7 messages.

There is also another issue that happens as a result of this one. You try to delete the message, but in the Android app, it gets deleted, and then it comes right back (it does show gone on the web/electron app, though).

@bkraul
Copy link

bkraul commented Mar 3, 2019

Any updates on this? This appears to happen more when on mobile data. It pretty much renders the app unusable.

@bizzbyster
Copy link
Contributor

As the steps to reproduce are written, I'm not able to make it happen. Maybe there's something I can do to exacerbate the problem so I can experience it?

@gigawhitlocks
Copy link

I suspect it appears more readily when the network connection from the client is not good. If you have a testing instance of RocketChat available, you could use the tc tool in Linux on that instance to simulate a shoddy connection to the listening port on the server.

@bizzbyster
Copy link
Contributor

can you please test the fix I just posted? I did eventually reproduce it but it would be great if you could verify that the fix addresses what you are seeing as well. Thanks!

@gigawhitlocks
Copy link

Yes, I will test this fix ASAP, hopefully tonight.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants