liqd / a4-meinberlin

The central participation platform of the city of Berlin, Germany
https://mein.berlin.de
GNU Affero General Public License v3.0
40 stars 6 forks source link

#8231 [mB] optimize image upload #5680

Closed mcastro-lqd closed 1 month ago

mcastro-lqd commented 1 month ago

URL: https://meinberlin-dev.liqd.net/dashboard/projects/project-0999/information/ user: initiator behaviour: In the story Caro mentioned

Add an file-size restriction of 5mb to CKEditor Image-Uploads (Here the error-message should be shown like the message when no alt-text is given to the image, but with the message “Das Bild darf höchstens 5 MB groß sein.”) - For mB wording (from weblate): “Image should be at most {max_size} MB” // “Das Bild darf höchstens {max_size} MB groß sein.”

However this is not happening cause I uploaded a 6mb pic and it worked out fine without error message

Screenshot 2024-08-01 at 11 51 00
pascalkienast commented 1 month ago

@mcastro-lqd thanks for the issue. i also noticed the same (and just wanted to write an issue as well) No matter how large the image (i also tryed an 6mb and even a 30mb image), everything worked and there was no error message.

goapunk commented 1 month ago

@mcastro-lqd @pascalkienast sorry, somehow the code didn't make it on the dev server as intended. It should work now, could you check again?

pascalkienast commented 1 month ago

@goapunk yes, now it seems to work. I would check it briefly in other browsers etc., but basically this issue can actually be closed now, as the problem is no longer so fundamental that it is not working at all

Bildschirmfoto 2024-08-05 um 09 40 42
pascalkienast commented 1 month ago

I have now tested this as initiator, admin, as well as under chrome, firefox, safari and via browserstack on mobile devices. It worked everywhere, so this issue can definitively be closed