8000 Update installation.mdx by sovetski · Pull Request #2590 · better-auth/better-auth · GitHub
[go: up one dir, main page]
Skip to content

Update installation.mdx #2590

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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and 8000 privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
May 9, 2025
Merged

Update installation.mdx #2590

merged 1 commit into from
May 9, 2025

Conversation

sovetski
Copy link
Contributor
@sovetski sovetski commented May 9, 2025

With the recent version of prisma, we can directly use a shared client code: https://www.prisma.io/docs/guides/nextjs#25-set-up-prisma-client

@sovetski
Update installation.mdx
2c90d4c 
With the recent version of prisma, we can directly use a shared client code
@jazzberry-ai Jazzberry AI
Copy link
jazzberry-ai bot commented May 9, 2025 •
edited
Loading

Bug Report

Incorrect import path.|High|Follow the installation 8000 instructions in the documentation, including the patched import statement. Run the application. Observe that the application fails to start due to the incorrect import path.|The documentation patch modifies the import statement for PrismaClient to an incorrect path. This will cause applications following the documentation to crash because the PrismaClient cannot be resolved. The documentation should be updated to reflect the correct import path, which is either node_modules/.prisma/client (if the client is not bundled) or a relative path to the generated client within the dist directory. It is also possible it should be @prisma/client, if the prisma generate command has been run.

Comments? Email us.

@vercel Vercel
Copy link
vercel bot commented May 9, 2025

@sovetski is attempting to deploy a commit to the better-auth Team on Vercel.

A member of the Team first needs to authorize it.

@pkg-pr-new pkg.pr.new
Copy link
pkg-pr-new bot commented May 9, 2025

Open in StackBlitz

npm i https://pkg.pr.new/better-auth/better-auth@2590

npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/cli@2590

npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/expo@2590

npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/stripe@2590

commit: 2c90d4c

mrge-io[bot]
mrge-io bot reviewed May 9, 2025
View reviewed changes
Copy link
@mrge-io mrge-io bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mrge found 1 issue across 1 file. Review it in mrge.io

@Bekacru Bekacru merged commit 7b2c498 into better-auth:main May 9, 2025
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mrge-io mrge-io[bot] mrge-io[bot] left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sovetski @Bekacru
0