Opened 11 years ago
Closed 11 years ago
#21675 closed New feature (fixed)
Provide an AppConfig for contrib apps with a translatable verbose_name
Reported by: | Aymeric Augustin | Owned by: | nobody |
---|---|---|---|
Component: | Core (Other) | Version: | dev |
Severity: | Normal | Keywords: | app-loading |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
That would make sense if we want to promote the use of AppConfig subclasses in INSTALLED_APPS.
It involves:
- picking good verbose names in English
- creating simple AppConfig modules
- updating the documentation
Change History (5)
comment:1 by , 11 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:2 by , 11 years ago
comment:3 by , 11 years ago
Setup code should likely by default include admin.site.autodiscover()
as this clearly does not belong in a urls.py
file.
comment:4 by , 11 years ago
Has patch: | set |
---|
https://github.com/django/django/pull/2145 creates AppConfig classes for all contrib apps.
The admin will use verbose_name for apps that have models, and admindocs too once #21685 is fixed.
For apps that don't have models, I don't think this change has any effect at this point.
But I included all apps in the patch for consistency. At least we have them if we introduce features that could take advantage of them.
This patch doesn't address the documentation; I don't know how much we want to promote usage of app configs; I'll discuss it on django-developers first. Even if we don't document it at all I think it's worth having these classes.
comment:5 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
When #21676 is done, we should try to move setup code in AppConfigs too. As is, it won't work for people still using the app name in INSTALLED_APPS, unless we reintroduce some kind of autodiscovery.