Skip to content

Add warning note for shadowDatabaseUrl.#4188

Merged
andrew-walford-prisma merged 1 commit intoprisma:mainfrom
rmariuzzo:patch-1
Dec 14, 2022
Merged

Add warning note for shadowDatabaseUrl.#4188
andrew-walford-prisma merged 1 commit intoprisma:mainfrom
rmariuzzo:patch-1

Conversation

@rmariuzzo
Copy link
Copy Markdown
Contributor

Describe this PR

As a developer I made the mistake to put the same value for both url and shadowDatabaseUrl. This lead to situation where the Prisma's migration table disappeared several times resulting in an inconsistent app state.

Changes

  • Added a note.

What issue does this fix?

Can be useful for other devs.

Any other relevant information

Probably, we could expand and tell concisely what happens when we put the same value.

As a developer I made the mistake to put the same value for both `url` and `shadowDatabaseUrl`. This lead to situation where the Prisma's migration table disappeared several times resulting in an inconsistent app state.
@vercel
Copy link
Copy Markdown

vercel bot commented Dec 5, 2022

@rmariuzzo is attempting to deploy a commit to the Prisma Team on Vercel.

A member of the Team first needs to authorize it.

@janpio
Copy link
Copy Markdown
Contributor

janpio commented Dec 5, 2022

I created an issue that we actually forbid that, so we will not need that warning when this is implemented: prisma/prisma#16628

@vercel
Copy link
Copy Markdown

vercel bot commented Dec 14, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
docs ❌ Failed (Inspect) Dec 14, 2022 at 10:59AM (UTC)

@andrew-walford-prisma
Copy link
Copy Markdown
Contributor

Even though we're going to fix the issue in the product, I've merged this docs PR, so that this information is available in the interrim. Thans @rmariuzzo

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.

3 participants