From 0056fb1d0fbbf17ab3d475181e1c63aa853e0418 Mon Sep 17 00:00:00 2001 From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com> Date: Tue, 29 Apr 2025 16:27:56 +0530 Subject: [PATCH] fix: require email OR phone in shipment doctype not both (backport #47300) (#47330) fix: require email OR phone in shipment doctype not both (#47300) (cherry picked from commit fc02a6510ee74191383107076c0b7fff3a1d0f91) Co-authored-by: Mihir Kandoi --- erpnext/stock/doctype/shipment/shipment.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/erpnext/stock/doctype/shipment/shipment.js b/erpnext/stock/doctype/shipment/shipment.js index 8843d383531..7f1e1c8d729 100644 --- a/erpnext/stock/doctype/shipment/shipment.js +++ b/erpnext/stock/doctype/shipment/shipment.js @@ -162,7 +162,7 @@ frappe.ui.form.on("Shipment", { args: { contact: contact_name }, callback: function (r) { if (r.message) { - if (!(r.message.contact_email && (r.message.contact_phone || r.message.contact_mobile))) { + if (!(r.message.contact_email || r.message.contact_phone || r.message.contact_mobile)) { if (contact_type == "Delivery") { frm.set_value("delivery_contact_name", ""); frm.set_value("delivery_contact", "");