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

Halt message-sending if one message fails to send #16771

Closed
ShadowJonathan opened this issue Mar 24, 2021 · 3 comments
Closed

Halt message-sending if one message fails to send #16771

ShadowJonathan opened this issue Mar 24, 2021 · 3 comments
Labels
A-Composer A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience T-Enhancement

Comments

@ShadowJonathan
Copy link
Contributor

ShadowJonathan commented Mar 24, 2021

This is mostly to avoid those "message failed to send" nightmare scenarios, where one specific message being left out of the message chain causes it to be interpreted incorrectly by the recipient(s), but this proposal is also to ensure a more robust delivery of messages.

(User Story: When my Wifi went down, the Element client kept on trying to send some messages out of a series i composed, however, when the wifi came back on, it only sent the last one, which confused my collegue before i showed them the other messages that failed to send.)

Halting when a message fails to send can be annoying, but in combination with #14181 could give better user experience by letting the messages only send in-order, exactly as you have submitted them, which gives a better perception of reliability for matrix.

Additionally, when a message fails to send, there should either be a button to "re-send", or "ignore" the error, this allows the user to control this error state, and if a message is particularly problematic (due to a bug), they could or should additionally also be able to report this (via a dedicated "report problem" button?), in any case, message-chaining will not continue until the message has either been successfully re-sent, or been explicitly ignored (to allow the next message(s) to send)

@ShadowJonathan
Copy link
Contributor Author

I recently also found out element does not cancel an 'edit' message transaction if the message that its editing is technically still unsent, creating an event that references a temporary transaction

@SimonBrandner SimonBrandner added O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience A-Composer labels Aug 22, 2021
@ShadowJonathan
Copy link
Contributor Author

Extra info; Telegram and Whatsapp are more robust with message handling (see element-hq/element-ios#4711), and almost never show a "cannot send message" error, except under network conditions which'd force them to send an image or so repeatedly, probably making the user burn up their data cap.

@MadLittleMods
Copy link
Contributor

Closing in favor of #22885 which has more participants

@MadLittleMods MadLittleMods closed this as not planned Won't fix, can't repro, duplicate, stale Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Composer A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience T-Enhancement
Projects
None yet
Development

No branches or pull requests

3 participants