-
Notifications
You must be signed in to change notification settings - Fork 63
Closed
Labels
status:doneTask is completeTask is complete
Description
Problem
PRs #630 (WA-NEW-006) and #631 (WA-NEW-007) are stacked on top of earlier commits (WA-NEW-002..005) that are not tracked as first-class issues/PRs in the current board/workflow. This makes review harder and blocks clean merge sequencing.
Goal
Make review/merge order clean and incremental by extracting WA-NEW-002..005 into their own issues + PRs (targeting next).
Scope
- Identify the commits currently bundled underneath WA-NEW-006: Fix Payment#complete? to treat zero authorization as incomplete #630/WA-NEW-007: Stabilize create_placed_order factory #631:
- WA-NEW-002 guard callback workers + truncate all Mongoid clients in tests
- WA-NEW-003 DataFile password randomization test determinism
- WA-NEW-004 metrics week math across DST
- WA-NEW-005 remove unsupported
use_dis_maxfrom query_string (ES7)
- Create missing tracking issues for each (WA-NEW-002..005) and add to board
- Create PRs for each (or rebase existing branches) so each PR is minimal and reviewable
- Rebase/retarget WA-NEW-006 and WA-NEW-007 PRs onto a
nextthat includes those earlier merges
Acceptance Criteria
- WA-NEW-002..005 each have: (1) issue, (2) PR, (3) board item
- WA-NEW-006: Fix Payment#complete? to treat zero authorization as incomplete #630 and WA-NEW-007: Stabilize create_placed_order factory #631 become minimal (only WA-NEW-006 and WA-NEW-007 changes)
- Merge order is explicit and linear
Notes
- This will also reduce the surface area when CI is restored (WA-NEW-012).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
status:doneTask is completeTask is complete