Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
|
There was a problem hiding this comment.
Pull Request Overview
This PR fixes TypeScript strict mode compatibility issues in order details page functionality by adding proper null checks and type annotations.
- Removed
@ts-strict-ignoredirective to enable strict TypeScript checking - Added comprehensive null safety guards throughout order data handling functions
- Enhanced type annotations for better type safety and null handling
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/orders/utils/data.ts | Added null checks, improved type safety, and removed TypeScript strict mode ignore directive |
| src/orders/components/OrderReturnPage/utils.tsx | Enhanced return type annotations and added explicit type parameters for reduce operations |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #5934 +/- ##
========================================
Coverage 40.06% 40.07%
========================================
Files 2473 2473
Lines 40171 40190 +19
Branches 8821 9171 +350
========================================
+ Hits 16096 16107 +11
- Misses 24048 24056 +8
Partials 27 27 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Scope of the change
This PR fixes error when opening order draft details. It also adds strict mode in a file