Skip to content

feat(functions): cron schedule field maps to cronPattern#286

Merged
kkopanidis merged 1 commit into
ui-rewritefrom
feat/functions-cron-ui
Jun 22, 2026
Merged

feat(functions): cron schedule field maps to cronPattern#286
kkopanidis merged 1 commit into
ui-rewritefrom
feat/functions-cron-ui

Conversation

@kkopanidis

Copy link
Copy Markdown
Contributor

Problem

The functions UI collected a cron schedule but stored it only as inputs.event. The backend now expects canonical inputs.cronPattern, and invalid patterns were accepted without client-side feedback.

Summary

  • Map cronStringinputs.cronPattern (keeps inputs.event as alias for transition)
  • Load edit forms from inputs.cronPattern ?? inputs.event
  • Client-side 5-field cron validation via cron-parser
  • Helper text: minute hour day month weekday (UTC)

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

The PR fulfills these requirements:

  • It's submitted to the ui-rewrite branch

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature

Test plan

  • Create cron function with */5 * * * *; confirm API payload includes inputs.cronPattern
  • Submit invalid cron (e.g. not a cron); confirm inline validation error
  • Edit an existing cron function stored with legacy inputs.event; confirm field pre-fills
  • Edit schedule and save; confirm updated pattern persists

Create and edit forms now send inputs.cronPattern, validate 5-field UTC
schedules client-side, and load legacy inputs.event values on edit.
@kkopanidis kkopanidis force-pushed the feat/functions-cron-ui branch from d34a1fc to 00a9a5d Compare June 22, 2026 07:49
@kkopanidis kkopanidis merged commit 491c324 into ui-rewrite Jun 22, 2026
2 of 3 checks passed
@kkopanidis kkopanidis deleted the feat/functions-cron-ui branch June 22, 2026 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant