Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#22770 closed Cleanup/optimization (fixed)

Remove create_superuser from post_migrate signals

Reported by: Tim Graham Owned by: nobody
Component: contrib.auth Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

In IRC, Loic and I were in agreement that the prompt to create a superuser every time after you run migrate can be annoying. Running createsuperuser manually is not much of a burden if you do need it. I think we could make the change without a deprecation as it's not clear to me that doing so would offer any benefits.

Change History (6)

comment:1 by Tim Graham, 10 years ago

Has patch: set
Triage Stage: UnreviewedAccepted

PR (proposing to include it in 1.7).

comment:2 by Michael Manfre, 10 years ago

If this type of change was proposed by a non core dev, would it even be considered given that 1.7 is already late in the beta stage?

comment:3 by Tim Graham, 10 years ago

I added the logic back to syncdb for backwards compatibility. Making the change now means we don't have to worry about people expecting the behavior with migrate.

comment:4 by Tim Graham <timograham@…>, 10 years ago

Resolution: fixed
Status: newclosed

In 93d05536fd8179483c725467de8e33454b48a430:

Fixed #22770 -- Removed create_superuser from post_migrate signals.

Moved logic to syncdb command for backwards compatibility.

comment:5 by Tim Graham <timograham@…>, 10 years ago

In 0767055dfc255ba120811f6d9cba2c7e593b0731:

[1.7.x] Fixed #22770 -- Removed create_superuser from post_migrate signals.

Moved logic to syncdb command for backwards compatibility.

Backport of 93d05536fd from master

comment:6 by Michael Angeletti, 10 years ago

I created https://github.com/django/django/pull/3143 to address the single remaining documentation straggler.

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