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 RemovedInDjango41Warning on default_app_config #193

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

jonahgeorge
Copy link

Since Django 3.2, the property has been deprecated as it is automatically detected by Django. Currently when django-address is loaded into a Django 3.2+ project, it throws deprecation warnings:

django.utils.deprecation.RemovedInDjango41Warning: 'address' defines default_app_config = 'address.apps.AddressConfig'. Django now detects this configuration automatically. You can remove default_app_config

The pattern in this PR can be found in other libraries that need to support wide ranges of Django versions:
https://github.com/revsys/django-health-check/blob/master/health_check/db/__init__.py

```
django.utils.deprecation.RemovedInDjango41Warning: 'address' defines default_app_config = 'address.apps.AddressConfig'. Django now detects this configuration automatically. You can remove default_app_config
```
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.

1 participant