Description of DEBUG setting in email is misleading if DEBUG == False.
|Reported by:||jMyles||Owned by:||jMyles|
|Severity:||Normal||Keywords:||debug, mail, errors, docstrings|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
If DEBUG == False, Django sends emails to those listed in the ADMIN setting with the exact text of the debug message.
At the bottom of this message is the following:
"You're seeing this error because you have DEBUG = True in your Django settings file. Change that to False, and Django will display a standard 500 page."
In the case that the message is displayed in an email, this content is misleading. Of course in order for the email to have been sent, we know that the user has DEBUG = False.
Thus, we need a short branch divorcing these two cases and displaying slightly different content.
Also (and please let me know if the convention is to separate these into two tickets) - there are some grammar issues with the docstrings - many need to be migrated to the present recurring tense / indicative mood if in fact that is going to remain the guideline.
Change History (6)
Changed 4 years ago by jMyles
comment:1 Changed 4 years ago by jMyles
- Needs documentation unset
- Needs tests unset
- Owner changed from nobody to jMyles
- Patch needs improvement unset
comment:2 Changed 4 years ago by anonymous
- Resolution set to needsinfo
- Status changed from new to closed
comment:5 Changed 2 years ago by nagasthya@…
- Resolution needsinfo deleted
- Status changed from closed to reopened