Pass extra HTML context through to "notify_responsible"
This commit is contained in:
parent
d48643319c
commit
b526d216ff
|
|
@ -298,6 +298,7 @@ def notify_responsible(
|
|||
instance,
|
||||
sender,
|
||||
content: NotificationBody = InvenTreeNotificationBodies.NewOrder,
|
||||
html_context: Optional[dict] = None,
|
||||
exclude=None,
|
||||
extra_users: Optional[list] = None,
|
||||
):
|
||||
|
|
@ -310,6 +311,7 @@ def notify_responsible(
|
|||
instance: The newly created instance
|
||||
sender: Sender model reference
|
||||
content (NotificationBody, optional): _description_. Defaults to InvenTreeNotificationBodies.NewOrder.
|
||||
html_context: Optional HTML context to include in the notification. Defaults to None.
|
||||
exclude (User, optional): User instance that should be excluded. Defaults to None.
|
||||
extra_users (list, optional): List of extra users to notify. Defaults to None.
|
||||
"""
|
||||
|
|
@ -323,7 +325,14 @@ def notify_responsible(
|
|||
if extra_users:
|
||||
users.extend(extra_users)
|
||||
|
||||
notify_users(users, instance, sender, content=content, exclude=exclude)
|
||||
notify_users(
|
||||
users,
|
||||
instance,
|
||||
sender,
|
||||
content=content,
|
||||
html_context=html_context,
|
||||
exclude=exclude,
|
||||
)
|
||||
|
||||
|
||||
def notify_users(
|
||||
|
|
@ -331,6 +340,7 @@ def notify_users(
|
|||
instance,
|
||||
sender,
|
||||
content: NotificationBody = InvenTreeNotificationBodies.NewOrder,
|
||||
html_context: Optional[dict] = None,
|
||||
exclude=None,
|
||||
):
|
||||
"""Notify all passed users or groups.
|
||||
|
|
@ -343,6 +353,7 @@ def notify_users(
|
|||
instance: The newly created instance
|
||||
sender: Sender model reference
|
||||
content (NotificationBody, optional): _description_. Defaults to InvenTreeNotificationBodies.NewOrder.
|
||||
html_context (dict, optional): Optional HTML context to include in the notification. Defaults to None.
|
||||
exclude (User, optional): User instance that should be excluded. Defaults to None.
|
||||
"""
|
||||
# Setup context for notification parsing
|
||||
|
|
@ -362,9 +373,11 @@ def notify_users(
|
|||
'template': {'subject': content.name.format(**content_context)},
|
||||
}
|
||||
|
||||
tmp = content.template
|
||||
if tmp:
|
||||
context['template']['html'] = tmp.format(**content_context)
|
||||
if tmp := content.template:
|
||||
html_context = html_context or {}
|
||||
html_context.update(content_context)
|
||||
|
||||
context['template']['html'] = tmp.format(**html_context)
|
||||
|
||||
# Create notification
|
||||
trigger_notification(
|
||||
|
|
|
|||
Loading…
Reference in New Issue