Skip to content
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 privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(error): Simplify P2024 message #1905

Merged
merged 2 commits into from
May 12, 2021
Merged

fix(error): Simplify P2024 message #1905

merged 2 commits into from
May 12, 2021

Conversation

janpio
Copy link
Contributor

@janpio janpio commented May 7, 2021 •
edited
Loading

@janpio
fix(error): Simplify P2024 message
bff6bec 
New docs page https://www.prisma.io/docs/concepts/components/prisma-client/working-with-prismaclient/connection-pool is the better place to learn about the connection pool and explain the timeout error and so on. Current message was considered misleading by some (e.g. prisma/prisma#6898)
@janpio janpio requested a review from pimeys May 7, 2021 21:10
@janpio janpio marked this pull request as ready for review May 7, 2021 21:10
@janpio janpio requested a review from tomhoule May 11, 2021 18:21
@tomhoule tomhoule added this to the 2.23.0 milestone May 12, 2021
@tomhoule tomhoule mentioned this pull request May 12, 2021
Closed
@tomhoule tomhoule merged commit 01fab2f into master May 12, 2021
@tomhoule tomhoule deleted the janpio/pooltimeout-error branch May 12, 2021 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomhoule tomhoule tomhoule approved these changes

@pimeys pimeys Awaiting requested review from pimeys

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

Successfully merging this pull request may close these issues.
2 participants
@janpio @tomhoule