-
Notifications
You must be signed in to change notification settings - Fork 3
fix(create-your-first-crud.mdx) : highlights corrected Step 4 #62
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
Conversation
…arch, PageAdminProjects.tsx
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
WalkthroughThe pull request enhances a tutorial on creating a CRUD application for managing projects using Prisma and tRPC. The changes introduce a comprehensive implementation of project management functionalities, including creating, reading, updating, and deleting projects. The tutorial now provides a more detailed guide with expanded database schema, tRPC router methods, and UI components for handling project-related operations. Changes
Sequence DiagramsequenceDiagram
participant Client
participant ProjectRouter
participant Database
Client->>ProjectRouter: getAll(cursor, limit, searchTerm)
ProjectRouter->>Database: Query projects
Database-->>ProjectRouter: Return paginated projects
ProjectRouter-->>Client: Send projects with nextCursor
Client->>ProjectRouter: create(projectData)
ProjectRouter->>Database: Insert new project
Database-->>ProjectRouter: Confirm creation
ProjectRouter-->>Client: Return created project
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 1
🧹 Nitpick comments (3)
pages/tutorials/create-your-first-crud.mdx (3)
Line range hint
4-9: Enhance the introduction section with prerequisites.The introduction provides a good overview but could benefit from explicitly listing the prerequisites and required knowledge.
Consider adding a "Prerequisites" section that lists:
- Required Node.js version
- Required knowledge of React, Next.js, and TypeScript
- Familiarity with tRPC and Prisma
- Any other tools or concepts needed
Line range hint
1-1200: Maintain consistent code styling throughout the tutorial.There are inconsistencies in the code examples:
- Mix of single and double quotes for imports
- Inconsistent use of filename annotations in code blocks
Consider:
- Using consistent quote style (preferably double quotes) throughout code examples
- Adding filename annotations to all code blocks for better context
- Using consistent import ordering
Line range hint
1-1200: Improve tutorial navigation and progress tracking.While the tutorial is well-structured, it could benefit from better navigation aids.
Consider adding:
- A table of contents at the beginning
- Estimated time for each step
- Progress indicators
- "Previous/Next" navigation links between steps
- A "Jump to Section" dropdown menu
###Changes
Highlights were not placed correctly on the lines that were updated in this section.
###Screenshots
On the left = how it was before
On the right = after the fix
Summary by CodeRabbit
New Features
UI Improvements