fix: resolve TS2769 errors in invoicePdfService by eliminating duplicate @types/react
Build & Deploy / Build & Push Docker Image (push) Successful in 1m9s
Test / Type Check (all packages) (push) Failing after 34s
Test / API Unit Tests (push) Has been skipped
Test / Homepage Unit Tests (push) Has been skipped
Test / Storefront Unit Tests (push) Has been skipped
Test / Admin Unit Tests (push) Has been skipped
Test / Dashboard Unit Tests (push) Has been skipped
Test / API Integration Tests (push) Has been skipped
Build & Deploy / Deploy to VPS (push) Successful in 3s
Build & Deploy / Build & Push Docker Image (push) Successful in 1m9s
Test / Type Check (all packages) (push) Failing after 34s
Test / API Unit Tests (push) Has been skipped
Test / Homepage Unit Tests (push) Has been skipped
Test / Storefront Unit Tests (push) Has been skipped
Test / Admin Unit Tests (push) Has been skipped
Test / Dashboard Unit Tests (push) Has been skipped
Test / API Integration Tests (push) Has been skipped
Build & Deploy / Deploy to VPS (push) Successful in 3s
Root cause: apps/api declared react@18 but @types/react@19, creating a nested apps/api/node_modules/@types/react@19.2.17 that conflicted with @react-pdf/renderer's type resolution from root node_modules/@types/react. Changes: - Removed react-dom from apps/api dependencies (unused) - Removed @types/react and @types/react-dom from apps/api devDependencies - Both API code and @react-pdf/renderer now resolve @types/react from a single root copy (v19), eliminating the TS2769 contextType incompatibility errors - Regenerated package-lock.json with deduped dependency tree Validation: - npm ls shows single @types/react installation - No nested @types/react under apps/api/node_modules - API type-check (tsc --noEmit): 0 errors - API build: succeeds - Docker build (Dockerfile.production): all 6 packages build successfully
This commit is contained in:
Generated
+2157
-7555
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user