#33971 closed Bug (fixed)
admindocs breaks the admin if it appears before the admin in INSTALLED_APPS
Reported by: | Tom Carrick | Owned by: | Mariusz Felisiak |
---|---|---|---|
Component: | contrib.admindocs | Version: | 4.1 |
Severity: | Release blocker | Keywords: | |
Cc: | Carlton Gibson, Alokik Roy | Triage Stage: | Accepted |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
Change History (8)
comment:1 Changed 9 months ago by
Summary: | admindocs doesn't work if it appears before the admin in INSTALLED_APPS → admindocs breaks the admin if it appears before the admin in INSTALLED_APPS |
---|
comment:2 Changed 9 months ago by
Description: | modified (diff) |
---|
comment:3 Changed 9 months ago by
Cc: | Carlton Gibson Alokik Roy added |
---|---|
Severity: | Normal → Release blocker |
Triage Stage: | Unreviewed → Accepted |
Version: | dev → 4.1 |
comment:4 Changed 9 months ago by
Has patch: | set |
---|---|
Owner: | changed from nobody to Mariusz Felisiak |
Status: | new → assigned |
comment:7 follow-up: 8 Changed 9 months ago by
Description: | modified (diff) |
---|
This patch war unnecessary.
with the same Effect we can remove method ready from class AdminDocsConfig(AppConfig) from django.contrib.admindocs.apps
For topik starter:
I use many times admindocs in my projects and it was easy solution: to redefine AdminDocsConfig with empty ready method.
comment:8 Changed 9 months ago by
Replying to Maxim Danilov:
This patch war unnecessary.
with the same Effect we can remove method ready from class AdminDocsConfig(AppConfig) from django.contrib.admindocs.apps
For topik starter:
I use many times admindocs in my projects and it was easy solution: to redefine AdminDocsConfig with empty ready method.
Removing ready()
was not a proper solution because admindocs
views wouldn't work without it as callbacks wouldn't be registered.
Thanks for the report! Unfortunately the proposed solution doesn't fix this for me, so we need to handle it separately.
Regression in 7f3cfaa12b28d15c0ca78bb692bfd6e59d17bff1.