[PR #29982] fix(schedule): support multiple schedule triggers in workflow #32637

Closed
opened 2026-02-21 20:51:47 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/29982

State: closed
Merged: No


Summary

  • Add extract_all_schedule_configs() method to return all schedule configs
  • Refactor sync_schedule_from_workflow() to handle multiple schedules
  • Follow same pattern as update_app_triggers handler for consistency

Test plan

  • Added tests for extract_all_schedule_configs() with multiple schedules
  • Added tests for sync operations (add/remove/update)
  • CI tests pass

Fixes #29945

**Original Pull Request:** https://github.com/langgenius/dify/pull/29982 **State:** closed **Merged:** No --- ## Summary - Add `extract_all_schedule_configs()` method to return all schedule configs - Refactor `sync_schedule_from_workflow()` to handle multiple schedules - Follow same pattern as `update_app_triggers` handler for consistency ## Test plan - [x] Added tests for `extract_all_schedule_configs()` with multiple schedules - [x] Added tests for sync operations (add/remove/update) - [ ] CI tests pass Fixes #29945
yindo added the pull-request label 2026-02-21 20:51:47 -05:00
yindo closed this issue 2026-02-21 20:51:47 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#32637