Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add info about the max upload limit when upload fails #4037

Closed
rubo77 opened this issue Oct 15, 2018 · 6 comments
Closed

Add info about the max upload limit when upload fails #4037

rubo77 opened this issue Oct 15, 2018 · 6 comments

Comments

@rubo77
Copy link
Contributor

rubo77 commented Oct 15, 2018

If you upload a large file, you get the error message

Upload Failed
The file 'verylarge.mp3' exceeds this home server's size limit for uploads

It would help a lot to the user, if the message would tell you the upload limit in this case.

@richvdh
Copy link
Member

richvdh commented Oct 15, 2018

@rubo77
Copy link
Contributor Author

rubo77 commented Oct 15, 2018

not working in riot-web version: 0.16.0 with synapse "0.33.5.1"

max_upload_size: "10M"

I tried an 10.5MB file

@richvdh
Copy link
Member

richvdh commented Oct 15, 2018

@Half-Shot did I imagine this?

@Half-Shot
Copy link
Collaborator

@Half-Shot did I imagine this?

@richvdh The react-sdk PR is still awaiting review so this doesn't happen yet.

@Half-Shot
Copy link
Collaborator

I don't think this needs to be kept open since Riot has the means to tell you the limits, whereas synapse doesn't even generate that error.

@richvdh
Copy link
Member

richvdh commented Oct 16, 2018

yup, so this is a riot bug. thanks Half-Shot.

@richvdh richvdh closed this as completed Oct 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants