Skip to content

Improve workflow schedule UI with 15-min time picker and bi-weekly option#25

Merged
alari76 merged 2 commits intomainfrom
fix/code-review-security-fixes
Mar 9, 2026
Merged

Improve workflow schedule UI with 15-min time picker and bi-weekly option#25
alari76 merged 2 commits intomainfrom
fix/code-review-security-fixes

Conversation

@alari76
Copy link
Copy Markdown
Contributor

@alari76 alari76 commented Mar 9, 2026

Summary

  • Replace hourly <select> dropdown with native <input type="time" step={900}> for 15-minute increment scheduling
  • Add bi-weekly frequency option (encoded as */14 day-of-month cron)
  • Reorganize frequency selector into two rows: presets (Daily, Weekdays, Bi-weekly) and individual days (Mon–Sun)
  • Apply same updates to both Add and Edit workflow modals for consistency

Test plan

  • All 852 existing tests pass
  • 8 new tests for buildCron/parseCron with minutes and bi-weekly, toTimeValue/fromTimeValue, describeCron bi-weekly
  • Manually verify time picker allows 15-min increments in browser
  • Manually verify bi-weekly option produces correct cron and description

🤖 Generated with Claude Code

alari76 and others added 2 commits March 9, 2026 09:00
Replace the hourly select dropdown with a native time input (15-minute
increments) and reorganize frequency selector into preset/individual day
rows with a new bi-weekly option.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
~/repos is a symlink to /srv/repos. realpathSync resolved the repo
path but not REPOS_ROOT, causing the startsWith check to fail.
Resolve REPOS_ROOT at startup so both sides compare real paths.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@alari76 alari76 merged commit 07546eb into main Mar 9, 2026
2 checks passed
@alari76 alari76 deleted the fix/code-review-security-fixes branch March 10, 2026 05:10
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