#21060 closed New feature (fixed)

Refactor admin's autodiscover method to make it reusable

Reported by: jcatalan Owned by: jcatalan
Component: Utilities Version: 1.5
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

It would be nice to be able to reuse admin's autodiscover method to be able to use it for instance with tasks.py modules.

Change History (4)

comment:1 Changed 18 months ago by jcatalan

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 18 months ago by jcatalan

  • Has patch set

Here's the PR I made for this ticket.

comment:3 Changed 18 months ago by timo

  • Patch needs improvement set

Left comments on PR.

comment:4 Changed 18 months ago by Tim Graham <timograham@…>

  • Resolution set to fixed
  • Status changed from new to closed

In 6feb75129f42fdac751d0b79a2a005b4c0ad5c2d:

Fixed #21060 -- Refactored admin's autodiscover method to make it reusable.

We want to be able to use it for instance for discovering tasks.py modules
inside the INSTALLED_APPS.

This commit therefore moves the logic to autodiscover_modules method in
django.utils.module_loading.

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