Description
Priority
P2
Type / Area
refactor, backend
Suggested Complexity
Medium
Summary
Consolidate the two report-export service implementations.
Problem
src/reports/report-export.service.ts and src/modules/reports/report-export.service.ts both exist, risking divergent export logic.
Scope
In Scope
Identify which is wired into the active reports module
Merge unique logic
Remove the unused copy
Out of Scope
New export formats
Files
src/reports/report-export.service.ts src/modules/reports/report-export.service.ts
Acceptance Criteria
Single canonical service remains
Existing report export tests pass
npm run build succeeds
How to Test
npm run test -- reports
npm run build
PR Requirements
CI checks must pass
npm run test passes
Include screenshots of test results
Link issue using Closes #ISSUE_NUMBER
Project Links
Repository: https://github.com/Stellar-Uzima/Uzima-Backend Community: https://t.me/+qqHJ6Ufbfmk0NzY0
Description
Priority
P2
Type / Area
refactor, backend
Suggested Complexity
Medium
Summary
Consolidate the two report-export service implementations.
Problem
src/reports/report-export.service.ts and src/modules/reports/report-export.service.ts both exist, risking divergent export logic.
Scope
In Scope
Identify which is wired into the active reports module
Merge unique logic
Remove the unused copy
Out of Scope
New export formats
Files
src/reports/report-export.service.ts src/modules/reports/report-export.service.ts
Acceptance Criteria
Single canonical service remains
Existing report export tests pass
npm run build succeeds
How to Test
npm run test -- reports
npm run build
PR Requirements
CI checks must pass
npm run test passes
Include screenshots of test results
Link issue using Closes #ISSUE_NUMBER
Project Links
Repository: https://github.com/Stellar-Uzima/Uzima-Backend Community: https://t.me/+qqHJ6Ufbfmk0NzY0