-
Notifications
You must be signed in to change notification settings - Fork 877
DR-7161 - DOCS: Update Nuxt guide #7476
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
Dangerous URL checkNo absolute URLs to prisma.io/docs found. |
Redirect checkThis PR probably requires the following redirects to be added to static/_redirects:
|
|
WalkthroughA comprehensive rewrite of the Nuxt guide, transforming it from a brief setup overview into a complete end-to-end tutorial covering Nuxt + Prisma ORM integration with PostgreSQL, including schema definition, migrations, seeding, API routes, and UI implementation. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~22 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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 |
Deploying docs with
|
| Latest commit: |
4712949
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://729f0dcd.docs-51g.pages.dev |
| Branch Preview URL: | https://dr-7161-docs-guide-nuxt-upda.docs-51g.pages.dev |
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: 2
🤖 Fix all issues with AI agents
In `@content/800-guides/100-nuxt.mdx`:
- Line 72: Update the heading "Define your Prisma Schema" to sentence case by
lowercasing "Schema" (change to "Define your Prisma schema") to match the
repository's MDX style; locate the heading string in the 2.2 section/title and
replace it accordingly.
- Line 37: The heading "## 2. Install and Configure Prisma" violates
sentence-case rules; change it to "## 2. Install and configure Prisma" (only
capitalize product names like "Prisma ORM") so the heading uses sentence case;
locate the heading text "## 2. Install and Configure Prisma" in the file and
update it accordingly.
🧹 Nitpick comments (2)
content/800-guides/100-nuxt.mdx (2)
107-119:prisma migrate devalready generates the client — the separategeneratestep is redundant.
npx prisma migrate devrunsprisma generateautomatically after applying migrations. The explicitnpx prisma generateon line 118 is harmless but unnecessary. Consider either removing it or adding a brief note explaining it's only needed if you skip the migration step — otherwise readers may think both are always required.
316-330: Good next steps section — one small addition would help.Consider adding a link to the Prisma Postgres getting started guide or mentioning deployment considerations (e.g., environment variable setup for production). Readers finishing this tutorial will naturally wonder "how do I deploy this?" — giving them a breadcrumb here saves a trip to the search bar.


Summary by CodeRabbit