Skip to content

fix: include schedule and max_items in flows list output#629

Merged
chubes4 merged 1 commit intomainfrom
fix/flows-list-schedule-max-items
Mar 5, 2026
Merged

fix: include schedule and max_items in flows list output#629
chubes4 merged 1 commit intomainfrom
fix/flows-list-schedule-max-items

Conversation

@chubes4
Copy link
Member

@chubes4 chubes4 commented Mar 5, 2026

Summary

  • add schedule and max_items columns to wp datamachine flows list default output
  • derive schedule from each flow's scheduling_config (including cron expressions)
  • derive max_items from handler configs in flow_config and surface as handler=max_items pairs

Why

datamachine/get-flows already returns scheduling and handler config data, but the CLI list view hid it. This made it hard to audit manual schedules and max_items without running flows get per flow.

Validation

  • php -l inc/Cli/Commands/Flows/FlowsCommand.php

@chubes4 chubes4 force-pushed the fix/flows-list-schedule-max-items branch from 5896015 to 1b49fa6 Compare March 5, 2026 14:43
@github-actions
Copy link

github-actions bot commented Mar 5, 2026

Homeboy Results — data-machine

Tooling versions

  • Homeboy CLI: homeboy 0.56.0
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: unknown
  • Action: Extra-Chill/homeboy-action@v1

ℹ️ PR test scope resolved to full for compatibility with installed Homeboy CLI

lint (changed files only)

  • PHPCS: LINT SUMMARY: 3 errors, 10 warnings
  • Fixable: 10 | Files with issues: 1 of 1
  • PHPStan: PHPSTAN SUMMARY: 133 errors at level 5

Homeboy Action v1 — homeboy 0.56.0

@chubes4 chubes4 force-pushed the fix/flows-list-schedule-max-items branch from 1b49fa6 to b0a1a12 Compare March 5, 2026 14:55
@chubes4 chubes4 merged commit 580bb48 into main Mar 5, 2026
2 checks passed
@chubes4 chubes4 deleted the fix/flows-list-schedule-max-items branch March 5, 2026 15:13
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