Merge pull request #3719 from Budibase/fix/incoming-webhooks

Fix/incoming webhooks

packages/builder/cypress/integration/createAutomation.spec.js

@@ -35,19 +35,20 @@ context("Create a automation", () => {
     cy.contains("dog").click()
     cy.get(".spectrum-Textfield-input")
     .first()
-    .type("goodboy")
+    .type("{{ trigger.row.name }}", { parseSpecialCharSequences: false })
     cy.get(".spectrum-Textfield-input")
       .eq(1)
       .type("11")
+    cy.contains("Finish and test automation").click()
 
-    cy.contains("Run test").click()
     cy.get(".modal-inner-wrapper").within(() => {
       cy.wait(1000)
       cy.get(".spectrum-Picker-label").click()
       cy.contains("dog").click()
+      cy.wait(1000)
       cy.get(".spectrum-Textfield-input")
         .first()
-        .type("goodboy")
+        .type("automationGoodboy")
       cy.get(".spectrum-Textfield-input")
         .eq(1)
         .type("11")
@@ -57,6 +58,9 @@ context("Create a automation", () => {
       cy.get(".spectrum-Textfield-input")
         .eq(3)
         .type("123456")
+      cy.contains("Test").click()
     })
+    cy.contains("Data").click()
+    cy.contains("automationGoodboy")
   })
 })

packages/builder/src/components/automation/AutomationBuilder/FlowChart/TestDataModal.svelte

@@ -25,7 +25,7 @@
 
   // Check the schema to see if required fields have been entered
   $: isError = !trigger.schema.outputs.required.every(
-    required => testData[required]
+    required => testData[required] || required !== "row"
   )
 
   function parseTestJSON(e) {

packages/builder/src/components/automation/SetupPanel/AutomationBlockSetup.svelte

@@ -244,7 +244,7 @@
             value={inputData[key]}
           />
         </CodeEditorModal>
-      {:else if value.type === "string" || value.type === "number"}
+      {:else if value.type === "string" || value.type === "number" || value.type === "integer"}
         {#if isTestModal}
           <ModalBindableInput
             title={value.title}

packages/server/src/api/controllers/webhook.js

@@ -65,6 +65,10 @@ exports.buildSchema = async ctx => {
     let automation = await db.get(webhook.action.target)
     const autoOutputs = automation.definition.trigger.schema.outputs
     let properties = webhook.bodySchema.properties
+    // reset webhook outputs
+    autoOutputs.properties = {
+      body: autoOutputs.properties.body,
+    }
     for (let prop of Object.keys(properties)) {
       autoOutputs.properties[prop] = {
         type: properties[prop].type,