[8231] introduce additional limits for file/image upload size - Githubissues

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] introduce additional limits for file/image upload size #5667

Closed goapunk closed 1 month ago

goapunk commented 1 month ago

Describe your changes

update django-ckeditor-5 so we can restrict the file/image size there
add max resolution constraints for organization logos
refactor ParagraphForm to work with the new django-ckeditor-5 (same as in a+ now)

Testing:

upload a logo in the organisation settings (dashboard) -> http://127.0.0.1:8003/dashboard/organisations/liqd/settings/ with width < 650 and > 650
upload an image / file in the ckeditor (e.g. project information or result with size < 5mb and >5mb
test the dasboard functionality (creating chapters, paragraphs, moving them around) for the text review module (http://127.0.0.1:8003/dashboard/modules/text-review/document/)

Tasks

[x] PR name contains story or task reference
[x] Steps to recreate and test the changes
[ ] Documentation (docs and inline)
[ ] Tests (including n+1 and django_assert_num_queries where applicable)
[x] Changelog