Include site in notification mail context

Issue #76 resolved
Martin Pauly
created an issue

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)

  1. Log in to comment