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

User is not aware his photo is larger than the limit #179

Open
ruilopesm opened this issue Jun 11, 2023 · 1 comment
Open

User is not aware his photo is larger than the limit #179

ruilopesm opened this issue Jun 11, 2023 · 1 comment
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@ruilopesm
Copy link
Member

Every time a user tries to upload an image that is larger than the limit (2MB, right now) the API only returns a 422 and the user is not informed. We should send him an in-app popup notification saying, for instance, "The image is too large. The maximum size is 2MB"

@ruilopesm ruilopesm added bug Something isn't working good first issue Good for newcomers labels Jun 11, 2023
@nelsonmestevao
Copy link
Member

This is a good enough approach but we could remove the error by just checking the image size limit on the client and save the network travel 💡

@Afonso-santos Afonso-santos self-assigned this Sep 7, 2023
@Afonso-santos Afonso-santos removed their assignment Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants