fix(checks): de-duplicate 051 migration prefix and thread AuditCtx through QA seed

David commented

2026-06-17 00:03:46 +02:00

Owner

The parallel merges that landed on main collided in two mechanical ways that the check.yml gate caught: three migrations all claimed prefix 051 (sqlx keys its ledger on the prefix, so the applied version was ambiguous), and the operator QA seed (src/modules/seed/qa.rs) had not been updated for the new AuditCtx parameter added to several project/time-tracking service methods nor for the new CreateProjectRequest.actual_end_date field, so the lib failed to compile.

Renumber the two later-added migrations to 052 (mileage_entries) and 053 (tax_rate_widen_column) in creation-time order, leaving 051 for payments_amount_positive, and update the doc comment in mileage_tracking/mod.rs that cited the old filename. Pass the existing system AuditCtx into create_project / create_project_phase / create_task / create_time_entry, construct a local system ctx for the create_work_type / create_task_status helpers, and set actual_end_date: None on both seeded (non-completed) QA projects.

#PMS-375

The parallel merges that landed on main collided in two mechanical ways that the check.yml gate caught: three migrations all claimed prefix 051 (sqlx keys its ledger on the prefix, so the applied version was ambiguous), and the operator QA seed (src/modules/seed/qa.rs) had not been updated for the new AuditCtx parameter added to several project/time-tracking service methods nor for the new CreateProjectRequest.actual_end_date field, so the lib failed to compile. Renumber the two later-added migrations to 052 (mileage_entries) and 053 (tax_rate_widen_column) in creation-time order, leaving 051 for payments_amount_positive, and update the doc comment in mileage_tracking/mod.rs that cited the old filename. Pass the existing system AuditCtx into create_project / create_project_phase / create_task / create_time_entry, construct a local system ctx for the create_work_type / create_task_status helpers, and set actual_end_date: None on both seeded (non-completed) QA projects. #PMS-375

David added 1 commit

2026-06-17 00:03:46 +02:00

fix(checks): de-duplicate 051 migration prefix and thread AuditCtx through QA seed

Check / fmt + clippy + compile + unit/doc tests (pull_request) Successful in 2m52s

Details

E2E / Playwright against staging (pull_request) Successful in 1m42s

Details

Integration / integration tests (pull_request) Successful in 7m58s

Details

Create release / Create release from merged PR (pull_request) Has been skipped

Details

47451a9b77

The parallel merges that landed on main collided in two mechanical ways that the check.yml gate caught: three migrations all claimed prefix 051 (sqlx keys its ledger on the prefix, so the applied version was ambiguous), and the operator QA seed (src/modules/seed/qa.rs) had not been updated for the new AuditCtx parameter added to several project/time-tracking service methods nor for the new CreateProjectRequest.actual_end_date field, so the lib failed to compile.

Renumber the two later-added migrations to 052 (mileage_entries) and 053 (tax_rate_widen_column) in creation-time order, leaving 051 for payments_amount_positive, and update the doc comment in mileage_tracking/mod.rs that cited the old filename. Pass the existing system AuditCtx into create_project / create_project_phase / create_task / create_time_entry, construct a local system ctx for the create_work_type / create_task_status helpers, and set actual_end_date: None on both seeded (non-completed) QA projects.

#PMS-375

vas2000-work commented

2026-06-17 02:13:58 +02:00

Owner

Superseded: the qa-seed compile fix + 051 migration-prefix de-duplication landed on main via #288 (kept 051_mileage_entries at 051, renumbered the two never-deployed migrations). Closing this duplicate.