Skip to content

feat(functions): cron timezone + stop dual-write event - #330

Merged
kkopanidis merged 2 commits into
ui-rewritefrom
cursor/feat-functions-cron-timezone-1507
Sep 13, 2026
Merged

feat(functions): cron timezone + stop dual-write event#330
kkopanidis merged 2 commits into
ui-rewritefrom
cursor/feat-functions-cron-timezone-1507

Conversation

@ChrisPdgn

Copy link
Copy Markdown

Summary

Aligns Admin Functions cron UI with Conduit #1507:

  • Add IANA timezone field (default UTC) on cron create/edit
  • Validate cron pattern in that timezone
  • Send inputs.cronPattern + inputs.timezone only (no dual-write to inputs.event)
  • Reset options when switching function type so Request↔Cron does not leave a broken form state

Base: ui-rewrite. Builds on merged #286 cron schedule → cronPattern.

Test plan

  • Create cron function with schedule + timezone (e.g. Europe/Athens) → API has cronPattern + timezone, no event
  • Edit cron → timezone loads; change timezone; save
  • Invalid timezone / invalid cron → form errors
  • Switch Request → Cron → Request → form still saves
  • Legacy cron with only inputs.event still loads schedule on edit

@ChrisPdgn
ChrisPdgn marked this pull request as ready for review September 11, 2026 15:09
Add IANA timezone on cron create/edit, validate schedule in that zone,
and stop dual-writing the pattern into inputs.event.
Avoid leaving cron-only options behind when switching away from cron,
which broke request/webhook/event validation after a type change.
@kkopanidis
kkopanidis force-pushed the cursor/feat-functions-cron-timezone-1507 branch from 101fcd5 to ab95da2 Compare September 13, 2026 11:33
@kkopanidis
kkopanidis merged commit 1ee3caf into ui-rewrite Sep 13, 2026
2 of 3 checks passed
@kkopanidis
kkopanidis deleted the cursor/feat-functions-cron-timezone-1507 branch September 13, 2026 11:33
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.

2 participants