LemmyNet / lemmy

🐀 A link aggregator and forum for the fediverse
https://join-lemmy.org
GNU Affero General Public License v3.0
13.13k stars 867 forks source link

"Reply" button can send multiple messages #706

Closed felipeforte closed 4 years ago

felipeforte commented 4 years ago

Currently, the "Reply" button has a probably not intended feature of sending multiple instances of the same message:

image

If the loading icon is pressed, it will send repeated messages.

Also, sometimes when I comment or send messages, I receive an error that I wasn't able to replicate again, but it read something along the lines of "Operation cancelled" in red, and apparently it still sent some extra cloned messages. Yesterday I probably spammed the hell out of @dessalines 's message inbox because of that hahaha

Using GNU IceCat under Arch Linux

dessalines commented 4 years ago

Ah yeah that's probably a bug, it should disable the button while loading. This probably needs to be done with all buttons.

I'm not sure about the operation cancelled thing, probably a DB locking issue that I can't fix until we finish some of the current federation work.