Skip to content

Schedules redesign 5/5: view page, list, and action modals#3552

Merged
tegan-temporal merged 23 commits into
feature/schedulesfrom
tegan/schedules-pr5-view
Jun 22, 2026
Merged

Schedules redesign 5/5: view page, list, and action modals#3552
tegan-temporal merged 23 commits into
feature/schedulesfrom
tegan/schedules-pr5-view

Conversation

@tegan-temporal

@tegan-temporal tegan-temporal commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Description & motivation 💭

Final PR of the stack. After this merges, feature/schedules is byte-identical to tegan/schedules-update (verified with an empty git diff).

  • New schedule-view/ card-based page: spec card with full-spec toggle, advanced settings, recent/upcoming workflow runs, workflow input, custom search attributes
  • schedule-action-modals/: trigger, backfill, and pause modals rendered through a conditionally-mounted host keyed on the confirmation-modal store, so every open starts with fresh state (no stale selections, reasons, or page-load timestamps). Trigger pre-selects Allow All (matching server behavior for manual triggers) and both overlap pickers mark the schedule's current policy
  • Backfill modal interprets entered times in the schedule's timezone (caption shows it) and disables confirm when end ≤ start
  • List components move to schedules-list/
  • Final cleanup: deletes the superseded view components, schedule-spec-label, legacy schedule types (ScheduleParameters family, Unspecified overlap policy), and legacy i18n keys

Testing 🧪

How was this tested 👻

  • Manual testing
  • E2E tests added
  • Unit tests added

pnpm check 0 errors, pnpm lint 0 errors, 2,080 unit tests passing, full integration suite 233 passed / 0 failed. New schedule-actions.spec.ts covers trigger patch bodies, backfill request + post-success modal usability, modal state reset on reopen, timezone caption, end-before-start validation, and delete. The e2e spec now does a full create → verify summary → delete loop but needs a run against a real Temporal server (not runnable locally).

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

On a schedule view page: trigger, backfill (try an end time before the start time, and reopen modals to confirm state resets), pause/unpause with a reason, and delete.

Checklists

Merge Checklist

🤖 Generated with Claude Code

@tegan-temporal
tegan-temporal requested a review from a team as a code owner June 11, 2026 18:10
@vercel

vercel Bot commented Jun 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
holocene Ready Ready Preview, Comment Jun 22, 2026 4:17pm

Request Review

@laurakwhit laurakwhit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good! Noticed a few small things, but no major blockers.

Comment thread src/lib/components/schedule/schedule-view/workflow-runs-recent.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/advanced-settings-card.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/schedule-view.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/schedule-view.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/workflow-input-card.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/schedule-spec-card.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/schedule-view-loading.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/schedule-view.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/workflow-runs-card.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/schedule-view.svelte Outdated

@laurakwhit laurakwhit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few super small comments, but otherwise :shipit:

Comment thread src/lib/components/schedule/schedule-view/schedule-view.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/schedule-view-loading.svelte Outdated
>
<Icon
name="filter"
title="Filter scheduled workflows by this type"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NIT: Add translation.

Comment thread src/lib/components/schedule/schedule-action-modals/schedule-action-modals.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/workflow-runs-recent.svelte Outdated
Comment thread src/lib/components/schedule/schedule-view/schedule-view.svelte Outdated
@tegan-temporal
tegan-temporal merged commit 637b8f7 into feature/schedules Jun 22, 2026
4 checks passed
@tegan-temporal
tegan-temporal deleted the tegan/schedules-pr5-view branch June 22, 2026 16:45
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