[🐛 Bug]: workflow trigger interval work just with int

[00041275]

syntax trigger for interval must be in 5m or cron, but we just can set int value.

docs/workflows/syntax/triggers.mdx

Interval Trigger

Runs workflows at a regular time interval or using a CRON expression.

triggers:
  - type: interval
    # Run every 5 minutes
    value: 5m

triggers:
  - type: interval
    # Run daily at 11:00 AM and 2:00 PM
    cron: "0 11,14 * * *"

keep/parser/parser.py

    def parse_interval(self, workflow) -> int:
        # backward compatibility
        workflow_interval = workflow.get("interval", 0)
        triggers = workflow.get("triggers", [])
        for trigger in triggers:
            if trigger.get("type") == "interval":
                workflow_interval = trigger.get("value", 0)
        return workflow_interval

[dosubot]

The interval trigger in the workflow is designed to accept integer values representing seconds. However, the documentation does provide examples using '5m' and cron syntax for interval triggers, indicating that these formats should be supported ^[1].

To convert expressions like '5m' into seconds, a function such as convert_to_seconds can be used, which multiplies the numeric part by the corresponding number of seconds per unit (e.g., 60 seconds per minute for 'm') ^[2].

It seems there might be a discrepancy between the documentation and the current implementation in parse_interval, which only retrieves integer values. You might need to ensure that the conversion logic is applied when parsing the interval values to support the documented syntax.

To continue talking to Dosu, mention @dosu.

---

^{Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other}

[00041275]

@talboren can you check this issue?

[Matvey-Kuk]

@00041275 thanks for the report! It's in issue in docs, there are no cron intervals.