Future of the COMPRESS_STORAGE
setting in Django 4.2+
#1199
Pinned
browniebroke
started this conversation in
Ideas
Replies: 1 comment
-
HI @browniebroke — thanks for this. Let's perhaps do this on #1187, which can be the tracking issue. I agree adding a My current half-idea would be to populate that with the Following Django's guidance, from the release of Django 5.0 onwards, I'd suggest dropping support for Django <4.2 if there's going to be any complications here. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Slightly related to #1187, but longer term idea.
Django recently deprecated the
STATICFILES_STORAGE
andDEFAULT_FILE_STORAGE
setting. In Cookiecutter Django, under some configuration, we doCOMPRESS_STORAGE = STATICFILES_STORAGE
. With the new setting, this would beCOMPRESS_STORAGE = STORAGES["staticfiles"]["BACKEND"]
, which doesn't read as well.So here is my question: should django-compressor consider adding or migrating the
COMPRESS_STORAGE
setting to something likeCOMPRESS_STORAGE_ALIAS
, which would refer to a key from theSTORAGES
dict?Obviously, this would only work for Django 4.2+, so it might make sense to wait before the old settings are completely gone from the versions supported by django-compressor, as I imagine reading from 2 possible settings * 2 values (default + user) might make the config code complicated.
Beta Was this translation helpful? Give feedback.
All reactions