Skip to content

feat(schedule-actions): add confirmation modal and mutation flow (PR2)#25

Closed
Assem-Uber wants to merge 1 commit into
feat/schedule-actions-pause-resume-uifrom
feat/schedule-actions-modal
Closed

feat(schedule-actions): add confirmation modal and mutation flow (PR2)#25
Assem-Uber wants to merge 1 commit into
feat/schedule-actions-pause-resume-uifrom
feat/schedule-actions-modal

Conversation

@Assem-Uber

@Assem-Uber Assem-Uber commented Jun 30, 2026

Copy link
Copy Markdown
Owner

Summary

Second PR in the schedule-actions UI stack. Adds the confirmation modal shell and modal content with config-driven warning banners, POST submission, query invalidation, snackbar, and error handling.

Stack (merge in order):

PR Scope
PR1 Types, config, menu (base branch)
PR2 (this) Modal + mutation flow
PR3 Page wiring

Depends on PR1 (cadence-workflow#1419). Not mounted on the schedule page until PR3.

Changes

  • schedule-actions-modal/ — modal wrapper; accepts optional schedule for banner render
  • schedule-actions-modal-content/ — renders modal.banner, confirm/cancel UI, mutation via request(), success snackbar, error banner

Testing

  • npm run typecheck
  • npm run test:unit:browser schedule-actions-modal

Add schedule-actions modal shell and modal content with config-driven
warning banners, POST submission, cache invalidation, snackbar, and
error handling.

Co-authored-by: Cursor <cursoragent@cursor.com>
@Assem-Uber Assem-Uber force-pushed the feat/schedule-actions-modal branch from 058bc54 to 566689a Compare June 30, 2026 13:59
@Assem-Uber Assem-Uber force-pushed the feat/schedule-actions-pause-resume-ui branch from 7ff39fa to 472b741 Compare June 30, 2026 13:59
@Assem-Uber

Copy link
Copy Markdown
Owner Author

Closing: stacked PRs should target cadence-workflow/cadence-web (upstream), not the fork. PR1 remains at cadence-workflow#1419.

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