#24944: Have password_reset pass extra_context to the email template rendering
as
well
------------------------------+---------------------------------------
Reporter: twelveeighty | Owner: SujaySKumar
Type: New feature | Status: new
Component: contrib.auth | Version: 1.8
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 1
Needs tests: 1 | Patch needs improvement: 1
Easy pickings: 1 | UI/UX: 0
------------------------------+---------------------------------------
Comment (by SujaySKumar):
Replying to [comment:4 timgraham]:
> A ticket isn't marked fixed until the patch is committed to Django. Also
your patch is missing tests and documentation as outlined in our
[https://docs.djangoproject.com/en/dev/internals/contributing/writing-code
/submitting-patches/#patch-review-checklist patch review checklist]. Also
your patch has a problem if someone is using `extra_context` with a key
that clashes with one in `opts` as I mentioned in comment 1.
I am new to contributing to open source. I am not able to understand where
to put the documentation for this new feature. I would be grateful if you
could point to the file in which I should add the documentation.
--
Ticket URL: <https://code.djangoproject.com/ticket/24944#comment:6>
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 post to this group, send email to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/070.19b0dc255eefcb547ce62302af8c7913%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
