fix(dashboard): replace any types with proper project data interfaces#312
fix(dashboard): replace any types with proper project data interfaces#312angelina-2206 wants to merge 1 commit into
Conversation
Signed-off-by: Angelina <angelinachatterjee2206@gmail.com>
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
👋 Thanks for opening a PR, @angelina-2206!Your PR has entered the 🚦 PR Review Pipeline.
What happens next
A pipeline status comment will appear below and update automatically as your PR progresses. While you wait
This comment is posted only once. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
WalkthroughTwo project table components tighten their TypeScript callback signatures. ProjectTable and CompactProjectTable now import ChangesType-safe project callback signatures
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
onUpdateProject,onDeleteProject, andonDuplicateProject) inProjectTableProps(modules/dashboard/components/project-table.tsx) andCompactProjectTableProps(modules/profile/components/CompactProjectTable.tsx) to return strongly-typed promises (Promise<void>andPromise<Playground>) instead ofPromise<any>.Playgroundmodel type from@prisma/clientto represent the duplication callback's return value.anytypes from the dashboard project table component interfaces, ensuring compile-time type safety and cleaner interface definitions.Type of change
Related issue
Closes #294
Validation
npm run lintnpm testnpm run buildList any additional manual verification you performed:
npx tsc --noEmit.Screenshots or recordings
(No UI changes)
Checklist
Summary by CodeRabbit