Skip to content

feat: integrate execute_queued_orders into campaign-qa pipeline#3

Open
travisd-tatari wants to merge 1 commit into
mainfrom
feat/execute-queued-orders-integration
Open

feat: integrate execute_queued_orders into campaign-qa pipeline#3
travisd-tatari wants to merge 1 commit into
mainfrom
feat/execute-queued-orders-integration

Conversation

@travisd-tatari
Copy link
Copy Markdown

Summary

  • Adds Step 20: Execute Queued Orders to campaign-qa — runs execute_queued_orders.py --latest after full form submission; skipped automatically for submit.action: draft
  • Updates Step 21: Report Results to include an Order Execution field
  • Adds error recovery entry #15 for order execution failures
  • Restores commands/bootstrap.md (accidentally dropped from HEAD in a previous commit)
  • Removes CM-9203 "not yet integrated" note from bootstrap.md final status

Related

  • Closes CM-9203
  • Companion commit in philo-fe/feat/campaign-qa-scenarios: all 13 scenarios updated submit.action: draftfull to enable end-to-end pipeline testing
  • execute_queued_orders.py authored by Nova (philo commit d950a2cf01) — this PR wires it into the automated command

Notes

  • submit.action: draft scenarios still work as before; Step 20 is a no-op
  • In local dev (EXTREME_REACH_ENABLED = False), ProgrammaticOrderRequest runs without needing ServeStreamingOrderRequest first — no streaming admin page required

- Add Step 20: Execute Queued Orders — runs execute_queued_orders.py
  --latest after full form submission (skipped for draft saves)
- Rename old Step 20 (Report Results) → Step 21
- Update report to include Order Execution result field
- Add error recovery entry #15 for order execution failures
- Restore commands/bootstrap.md (accidentally dropped from HEAD)
- Update bootstrap.md final status message: remove CM-9203 "not yet
  integrated" note, replace with guidance on full pipeline testing

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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