#34542: Required fields allowed to be blank are not accepted non-interactively
using createsuperuser
-------------------------------------+-------------------------------------
Reporter: Lantizia | Owner: Anvansh
| Singh
Type: Bug | Status: assigned
Component: contrib.auth | Version: dev
Severity: Normal | Resolution:
Keywords: auth | Triage Stage: Accepted
createsuperuser superuser email |
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by Mateusz Więckowski):
Replying to [comment:20 Natalia Bidart]:
> Hello Mateusz,
>
> Did you run the examples above with or without your changes? Using
Django main, this is what I get:
>
> {{{
> $ DJANGO_SUPERUSER_EMAIL="" python -Wall manage.py createsuperuser
--username testing --noinput
> CommandError: You must use --email with --noinput.
> }}}
>
> If you are confident that your proposal is correct and works for every
case, then please create a PR and I'll happily review. Thanks!
I ran the examples with my changes. I found one case with environmental
variables that wasn't working, but managed to fix it.
At the moment I am confident that my current solution works, but before
doing the PR I have just one question - should I write tests as well? In
the ticket details, it says 'Needs tests: no', but in the guide about
creating a PR, it's mentioned that tests are always needed, except for
documentation changes.
--
Ticket URL: <https://code.djangoproject.com/ticket/34542#comment:23>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/010701898da0f5ca-43e2db28-9115-42a4-9581-a918b72ab874-000000%40eu-central-1.amazonses.com.
