vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #27480 from akhilnarang/fix-notification-error-handling

Browse source 
fix(notification): can't use `message` where it isn't defined
This commit is contained in:
Akhil Narang 2024-08-22 11:04:27 +05:30 • committed by GitHub
commit 7e2a69a95b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
frappe/email/doctype/notification/notification.py
View file

@ -612,8 +612,8 @@ def evaluate_alert(doc: Document, alert, event):
frappe.throw(message, title=_("Error in Notification"))
except Exception as e:
title = str(e)
frappe.log_error(title=title)
message = frappe.get_traceback(with_context=True)
frappe.log_error(title=title, message=message)
msg = f"<details><summary>{title}</summary>{message}</details>"
frappe.throw(msg, title=_("Error in Notification"))