Refactor TODOs for licensing and maintainability
Added a critical pre-commercial release TODO to remove `SixLabors.ImageSharp` and `QRCoder-ImageSharp` dependencies and replace them with permissive-license alternatives. Updated TODOs to focus on modularizing `EnvelopeSenderPage.razor` and other server pages by splitting large components, centralizing shared logic, and improving maintainability. Included a TODO for a senior-level performance review of sender/receiver flows to address route transitions, loading logic, and perceived latency issues. Removed outdated TODO on reducing `EnvelopeSenderPage.razor` complexity.
This commit is contained in:
@@ -268,6 +268,7 @@ Manual testing workflow:
|
||||
|
||||
## Technical Debt TODOs
|
||||
|
||||
- **CRITICAL (Licensing / pre-commercial release):** Remove all `SixLabors.ImageSharp` and `QRCoder-ImageSharp` package dependencies from production/runtime projects before commercial release. Replace remaining QR/image paths with permissive-license alternatives (no copyleft/share-source obligations), run repo-wide package audit (`*.csproj`), and add CI guardrails to block reintroduction.
|
||||
- Refactor `EnvelopeGenerator.Server/EnvelopeGenerator.Server/Components/Pages/EnvelopeSenderPage.razor` to reduce complexity.
|
||||
- Move sender dashboard business/status logic into dedicated services and extension methods (for example effective status resolution, tab classification, receiver signed/rejected detection, and grid-layout filter sanitation).
|
||||
- Keep `EnvelopeSenderPage.razor` focused on UI composition/state orchestration; avoid embedding heavy domain logic directly in the Razor component.
|
||||
|
||||
Reference in New Issue
Block a user