- edited description
Include site in notification mail context
Issue #76
resolved
Hi, thanks for developing postman!
in the notifications mail send to users it would be good to have access to the absolute url of the page. Normally I would fetch this url from the request object, but here in the mail I do not have access to the request. However I believe that the site object in the notify_user function contains the information that I need for the mail context.
Thus it would be great if in https://bitbucket.org/psam/django-postman/src/74bf484632f8e4882dc249dc193b08bab6bb37b2/postman/utils.py?at=default&fileviewer=file-view-default#utils.py-99 the site variable could be passed in the extra_context dict as well.
Comments (3)
-
reporter -
reporter I added a Pull Request to add the object to the context
-
repo owner - changed status to resolved
PR accepted.
- Log in to comment