Browse Source

fix cron trigger output

master
Martin McKeaveney 5 years ago
parent
commit
0340c735e8
  1. 5
      packages/server/src/api/controllers/automation.js
  2. 10
      packages/server/src/automations/triggers.js

5
packages/server/src/api/controllers/automation.js

@ -65,7 +65,10 @@ async function checkForCronTriggers({ appId, oldAuto, newAuto }) {
// need to create cron job
else if (isCronTrigger(newAuto) && cronTriggerActivated) {
const job = await triggers.automationQueue.add(
{ automation: newAuto, event: { appId } },
{
automation: newAuto,
event: { appId, timestamp: Date.now() },
},
{ repeat: { cron: newTrigger.inputs.cron } }
)
// Assign cron job ID from bull so we can remove it later if the cron trigger is removed

10
packages/server/src/automations/triggers.js

@ -216,7 +216,15 @@ const BUILTIN_DEFINITIONS = {
},
required: ["cron"],
},
outputs: {},
outputs: {
properties: {
timestamp: {
type: "number",
description: "Timestamp the cron was executed",
},
},
required: ["timestamp"],
},
},
type: "TRIGGER",
},

Loading…
Cancel
Save