Opened 5 years ago

Closed 5 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: master
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


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 Changed 5 years ago by Aymeric Augustin

Triage Stage: UnreviewedAccepted

comment:2 Changed 5 years ago by Aymeric Augustin

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.

comment:3 Changed 5 years ago by Marc Tamlyn

Setup code should likely by default include as this clearly does not belong in a file.

comment:4 Changed 5 years ago by Aymeric Augustin

Has patch: set 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 Changed 5 years ago by Aymeric Augustin <aymeric.augustin@…>

Resolution: fixed
Status: newclosed

In 27afd302c687528ff1dc3156961e409862c6eca2:

Fixed #21675 -- Added app configs for contrib apps.

Note: See TracTickets for help on using tickets.
Back to Top