Skip to content

feat(returns): canonical consolidation of returns/refunds controls (supersedes #14, #18)#19

Closed
rmak78 wants to merge 1 commit intomainfrom
feat/returns-refunds-canonical-pr14-pr18
Closed

feat(returns): canonical consolidation of returns/refunds controls (supersedes #14, #18)#19
rmak78 wants to merge 1 commit intomainfrom
feat/returns-refunds-canonical-pr14-pr18

Conversation

@rmak78
Copy link
Owner

@rmak78 rmak78 commented Feb 27, 2026

Summary

This PR creates the canonical returns/refunds consolidation branch and supersedes overlapping work in #14 and #18.

What this keeps

  • Reason-code support for returns (return_reason_codes, sales_returns.return_reason_code)
  • Approval workflow fields on returns (approval_status, approved_by_employee_id, approved_at)
  • Refund method tracking + status indexing on sales_refunds
  • Existing inventory reversal behavior remains unchanged in API flow (sales-return-lines restock path)

Implementation

  • Adds one migration: data-model/migrations/0011_returns_refunds_controls.sql
  • Uses ALTER TABLE on existing baseline returns/refunds tables (no duplicate replacement tables)
  • Adds indexes for approval/reason/status query paths

Validation

  • npx wrangler deploy --dry-run

Supersedes

Please close #14 and #18 after this PR is accepted as the canonical path.

@rmak78 rmak78 force-pushed the feat/returns-refunds-canonical-pr14-pr18 branch from f59f457 to fbbeb7f Compare February 27, 2026 11:40
@rmak78
Copy link
Owner Author

rmak78 commented Feb 27, 2026

Superseded by #20 (clean canonical consolidation PR for returns/refunds).

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