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

Fix compatibility with Django 5.1 #180

Merged
merged 1 commit into from
Mar 27, 2024
Merged

Fix compatibility with Django 5.1 #180

merged 1 commit into from
Mar 27, 2024

Conversation

washeck
Copy link
Contributor

@washeck washeck commented Mar 27, 2024

Do not import get_storage_class() which is deprecated since Django 4.2. 2048348 stopped using it on recent Django version but did not remove the import so sass_processor stopped working on the upcoming Django 5.1 which removes the get_storage_class() function.

Do not import get_storage_class() which is deprecated since
Django 4.2. 2048348 stopped using it on recent Django version
but did not remove the import so sass_processor stopped working
on the upcoming Django 5.1 which removes the get_storage_class()
function.
@washeck
Copy link
Contributor Author

washeck commented Mar 27, 2024

I thought this would be covered by my previous commit (warnings catch by tests) but it did not as the DeprecationWarning is raised by Django on get_storage_class call not at the import time.

Anyway, I am testing compatibility of our project with main branch of Django (the upcoming 5.1) and this is blocking me.

@jrief jrief merged commit de8fdcb into jrief:master Mar 27, 2024
11 checks passed
@washeck washeck deleted the django51 branch March 27, 2024 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants