-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat: base node #2168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
feat: base node #2168
Conversation
006c7d3 to
822896e
Compare
5604895 to
aec87dc
Compare
|
Deployment Status: ✅ Success |
7fd67fe to
e0d3789
Compare
|
Deployment Status: ✅ Success |
6 similar comments
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
d6fcbce to
4d4e19b
Compare
|
Deployment Status: ✅ Success |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR introduces a hierarchical base node system to organize tables, dashboards, workflows, and apps within folders. It adds tree navigation capabilities using the @headless-tree library, implements comprehensive CRUD APIs for node management, and includes database models with appropriate migrations.
Key Changes
- Adds
@headless-tree/coreand@headless-tree/react(v1.5.1) for tree functionality - Implements BaseNode and BaseNodeFolder database models with self-referential relationships
- Creates comprehensive base node API endpoints (CRUD, move, duplicate operations)
- Adds Tree UI component with drag-and-drop support and folder depth validation
- Updates pin functionality to support new resource types (Dashboard, Workflow, App)
Reviewed changes
Copilot reviewed 119 out of 120 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-lock.yaml, packages/ui-lib/package.json | Adds @headless-tree dependencies v1.5.1 |
| packages/db-main-prisma/prisma/*.prisma | Defines BaseNode and BaseNodeFolder models with constraints |
| packages/openapi/src/base-node/* | Implements base node API schemas and routes |
| packages/ui-lib/src/shadcn/ui/tree.tsx | New Tree UI component with item management |
| apps/nextjs-app/src/features/app/blocks/base/base-node/* | React hooks and context for node management |
| apps/nestjs-backend/test/base-node*.e2e-spec.ts | Comprehensive e2e tests for node operations |
| packages/openapi/src/pin/types.ts | Extends PinType enum with Dashboard, Workflow, App |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
apps/nextjs-app/src/features/app/blocks/base/base-node/hooks/useBaseNode.ts
Outdated
Show resolved
Hide resolved
apps/nextjs-app/src/features/app/blocks/space/space-side-bar/PinItem.tsx
Outdated
Show resolved
Hide resolved
apps/nextjs-app/src/features/app/blocks/space/space-side-bar/PinItem.tsx
Outdated
Show resolved
Hide resolved
|
Deployment Status: ✅ Success |
2 similar comments
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
a8be369 to
c5dfe48
Compare
|
Deployment Status: ✅ Success |
2 similar comments
|
Deployment Status: ✅ Success |
|
Deployment Status: ✅ Success |
4182efd to
f61c9cc
Compare
|
Deployment Status: ✅ Success |
1 similar comment
|
Deployment Status: ✅ Success |
…n BaseNodeTree component
…tured response objects
…ding in BaseNodeTree
…e navigation in BaseNodeTree
…sponsiveness and accessibility
9ddece8 to
cef4ec5
Compare
No description provided.