#22752: PasswordResetForm email context is missing current_app
--------------------------------------+------------------------------------
Reporter: bendavis78 | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: contrib.auth | Version: master
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 1
Needs tests: 1 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
--------------------------------------+------------------------------------
Changes (by bmispelon):
* needs_better_patch: 0 => 1
* needs_tests: 0 => 1
* needs_docs: 0 => 1
* has_patch: 0 => 1
* type: Uncategorized => Cleanup/optimization
* stage: Unreviewed => Accepted
Comment:
Hi,
I agree that it would be good to have the current app while rendering the
body of the email because it would help with reversing URLs inside it.
However, I don't really understand the use-case behind having the current
app when rendering the subject line.
In any case, the provided patch doesn't apply on master and it's also
going to require tests and documentation.
Thanks.
--
Ticket URL: <https://code.djangoproject.com/ticket/22752#comment:3>
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/068.66ef2985cb23052b0a35e470cc8f5785%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
