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

perf: cache notifications for 1 minute

Browse source 
+ SWR for 5 minutes
This commit is contained in:
Ankush Menat 2025-01-23 12:04:31 +05:30
parent 378b638d34
commit f177bb28a7
2 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
frappe/desk/doctype/notification_log/notification_log.py
View file

@ -8,6 +8,7 @@ from frappe.desk.doctype.notification_settings.notification_settings import (
is_notifications_enabled, is_notifications_enabled,
) )
from frappe.model.document import Document from frappe.model.document import Document
from frappe.utils.caching import http_cache
class NotificationLog(Document): class NotificationLog(Document):
@ -164,6 +165,7 @@ def get_email_header(doc, language: str | None = None):
@frappe.whitelist() @frappe.whitelist()
@http_cache(max_age=60, stale_while_revalidate=5 * 60)
def get_notification_logs(limit=20): def get_notification_logs(limit=20):
notification_logs = frappe.db.get_list( notification_logs = frappe.db.get_list(
"Notification Log", fields=["*"], limit=limit, order_by="creation desc" "Notification Log", fields=["*"], limit=limit, order_by="creation desc"

1
frappe/public/js/frappe/ui/notifications/notifications.js
View file

@ -329,6 +329,7 @@ class NotificationsView extends BaseNotificationsView {
method: "frappe.desk.doctype.notification_log.notification_log.get_notification_logs", method: "frappe.desk.doctype.notification_log.notification_log.get_notification_logs",
args: { limit: limit }, args: { limit: limit },
type: "GET", type: "GET",
cache: true,
}); });
} }