Skip to content

No changes needed - keep custom Prisma import path#30

Merged
HpatricioH merged 1 commit into
developfrom
copilot/sub-pr-27-another-one
Dec 16, 2025
Merged

No changes needed - keep custom Prisma import path#30
HpatricioH merged 1 commit into
developfrom
copilot/sub-pr-27-another-one

Conversation

Copilot AI commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

Fixes Issue

Changes Proposed

No code changes made. The review comment suggested changing the Prisma client import from "../../prisma/prisma/client" to "@prisma/client" in src/server/db.ts, but the maintainer (@HpatricioH) explicitly declined this change, confirming the custom path is intentional for this project.

The existing import structure is maintained:

import { PrismaClient } from "../../prisma/prisma/client";

Checklist (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • The title of my pull request is a short description of the requested changes.

Screenshots


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@vercel

vercel Bot commented Dec 16, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
slug-link Ready Ready Preview, Comment Dec 16, 2025 6:02am

Copilot AI changed the title [WIP] Address feedback on Next upgrade to prevent vulnerabilities No changes needed - keep custom Prisma import path Dec 16, 2025
Copilot AI requested a review from HpatricioH December 16, 2025 06:01
@HpatricioH HpatricioH marked this pull request as ready for review December 16, 2025 06:02
@HpatricioH HpatricioH merged commit 1a4159c into develop Dec 16, 2025
3 checks passed
@HpatricioH HpatricioH deleted the copilot/sub-pr-27-another-one branch May 16, 2026 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants