fix: remove regions from project cards by chronark · Pull Request #5276 · unkeyed/unkey
📝 Walkthrough
Walkthrough
The changes remove regions tracking functionality across the dashboard project system by eliminating the regions prop from project cards, deleting the RegionsBadges component, removing the regions field from the Project schema and TRPC resolver output, and removing regions from new project creation payloads.
Changes
| Cohort / File(s) | Summary |
|---|---|
UI Components web/apps/dashboard/app/(app)/[workspaceSlug]/projects/_components/list/index.tsx, web/apps/dashboard/app/(app)/[workspaceSlug]/projects/_components/list/projects-card.tsx |
Removed regions prop from ProjectsList mapping and ProjectCardProps type. Reformatted InfoTooltip usage and adjusted layout structure in ProjectCard. |
Deleted Component web/apps/dashboard/app/(app)/[workspaceSlug]/projects/_components/list/region-badges.tsx |
Removed RegionBadges component that previously rendered region badges, remaining region tooltips, and repository display for project cards. |
Data Model & API web/apps/dashboard/lib/collections/deploy/projects.ts, web/apps/dashboard/lib/trpc/routers/deploy/project/list.ts |
Removed regions field from Project schema and removed regions property from listProjects resolver output. |
Project Creation web/apps/dashboard/app/(app)/[workspaceSlug]/projects/new/steps/create-project.tsx |
Removed regions array field from new project insertion payload. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Description check | ⚠️ Warning | The PR description lacks required sections including linked issue reference, type of change selection, testing instructions, and completion of the provided checklist. | Add the missing required sections: reference an issue number, select the type of change (appears to be a Bug fix), provide testing instructions, and complete the checklist items to confirm prerequisites. |
✅ Passed checks (2 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | The title 'fix: remove regions from project cards' clearly and concisely describes the main change—removing the regions feature from project card components across multiple files. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
- 📝 Generate docstrings (stacked PR)
- 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Post copyable unit tests in a comment
- Commit unit tests in branch
project-card-regions
Comment @coderabbitai help to get the list of available commands and usage tips.