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

Merge pull request #8548 from ruchamahabal/fix_webhook

Browse source 
fix: include table fields in webhook data
This commit is contained in:
mergify[bot] 2019-10-04 07:06:45 +00:00 • committed by GitHub
commit e32efa295e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
frappe/integrations/doctype/webhook/webhook.js
View file

@ -7,7 +7,7 @@ frappe.webhook = {
frappe.model.with_doctype(frm.doc.webhook_doctype, () => {
// get doctype fields
let fields = $.map(frappe.get_doc("DocType", frm.doc.webhook_doctype).fields, (d) => {
if (frappe.model.no_value_type.includes(d.fieldtype) || frappe.model.table_fields.includes(d.fieldtype)) {
if (frappe.model.no_value_type.includes(d.fieldtype) && !(frappe.model.table_fields.includes(d.fieldtype))) {
return null;
} else if (d.fieldtype === 'Currency' || d.fieldtype === 'Float') {
return { label: d.label, value: d.fieldname };