From 94263bf4f21ea15c7087717a50c313643d9f7fbe Mon Sep 17 00:00:00 2001 From: Sayed Ayman Date: Mon, 16 Oct 2023 09:32:19 +0300 Subject: [PATCH] feat (notificaion): email from 'Table Multiselect' field (#22733) * feat (notificaion): table multiselect field * refactor: use model.table_fields instead of duplicating --------- Co-authored-by: Ankush Menat --- frappe/email/doctype/notification/notification.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/email/doctype/notification/notification.js b/frappe/email/doctype/notification/notification.js index eb3e2e8634..5be70b14b0 100644 --- a/frappe/email/doctype/notification/notification.js +++ b/frappe/email/doctype/notification/notification.js @@ -50,7 +50,7 @@ frappe.notification = { if (frm.doc.channel === "Email") { receiver_fields = $.map(fields, function (d) { // Add User and Email fields from child into select dropdown - if (d.fieldtype == "Table") { + if (frappe.model.table_fields.includes(d.fieldtype)) { let child_fields = frappe.get_doc("DocType", d.options).fields; return $.map(child_fields, function (df) { return df.options == "Email" ||