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.

Sign up for GitHub

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

Jump to bottom

Descriptive error messages when media fails to send. #4749

Open
pixlwave opened this issue Aug 24, 2021 · 2 comments
Open

Descriptive error messages when media fails to send. #4749

pixlwave opened this issue Aug 24, 2021 · 2 comments
Labels
A-Composer Z-D1 Issues that can impact the Day-2 retention Z-Papercuts Visible. Impactful. Predictable to action.

Comments

@pixlwave
Copy link
Member

When a message fails to send, the standard text "Messages failed to send" is shown. Regarding videos, along with network issues, the failure could also be because of:

  • The encoder failed.
  • The media exceeds the homeserver's max upload size

We should add some more reasons for failure to properly communicate what went wrong, possibly falling back to "Messages failed to send" if more than one type of error occurred.

In this instance the video failed to encode, but it isn't obvious:
Simulator Screen Shot - iPhone 12 mini - 2021-08-24 at 16 33 41
@pixlwave pixlwave added A-Composer Z-Papercuts Visible. Impactful. Predictable to action. labels Aug 24, 2021
@pixlwave pixlwave mentioned this issue Aug 24, 2021
Open
@manuroe manuroe added the Z-D1 Issues that can impact the Day-2 retention label Oct 12, 2021
@daniellekirkwood daniellekirkwood added the X-Needs-Design May require input from the design team label Oct 18, 2021
@pixlwave
Copy link
Member Author

WIP Screen recording:

Failed.mov

@pixlwave
Copy link
Member Author

Figma link: https://www.figma.com/file/bGyPclOvERYA1Y0vIjMALr/[iOS]-Error-message-when-sending-message?node-id=0%3A1

@pixlwave pixlwave removed the X-Needs-Design May require input from the design team label Oct 19, 2021
This was referenced Oct 21, 2021
Merged
Closed
@pixlwave pixlwave self-assigned this Oct 21, 2021
@pixlwave pixlwave removed their assignment Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Composer Z-D1 Issues that can impact the Day-2 retention Z-Papercuts Visible. Impactful. Predictable to action.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Show the reason a message failed to send by tapping on it. element-hq/element-ios
3 participants
@pixlwave @manuroe @daniellekirkwood