Skip to content
This repository has been archived by the owner on Aug 30, 2023. It is now read-only.

Add referential actions' links for warnings handling Postgres and MySQL special cases #79

Merged
merged 3 commits into from Nov 28, 2022

Conversation

jkomyno
Copy link
Contributor

@jkomyno jkomyno commented Nov 25, 2022

@vercel
Copy link

vercel bot commented Nov 25, 2022

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

Name Status Preview Updated
pris-ly ✅ Ready (Inspect) Visit Preview Nov 25, 2022 at 4:03PM (UTC)

vercel.json Show resolved Hide resolved
vercel.json Outdated Show resolved Hide resolved
@Jolg42
Copy link
Member

Jolg42 commented Nov 25, 2022

Note: Alberto is waiting for his docs PR to be ready before merging this

Comment on lines +227 to +234
{
"source": "/d/postgres-set-null",
"destination": "https://www.prisma.io/docs/concepts/components/prisma-schema/relations/referential-actions#types-of-referential-actions"
},
{
"source": "/d/mysql-set-default",
"destination": "https://www.prisma.io/docs/concepts/components/prisma-schema/relations/referential-actions#types-of-referential-actions"
},
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
{
"source": "/d/postgres-set-null",
"destination": "https://www.prisma.io/docs/concepts/components/prisma-schema/relations/referential-actions#types-of-referential-actions"
},
{
"source": "/d/mysql-set-default",
"destination": "https://www.prisma.io/docs/concepts/components/prisma-schema/relations/referential-actions#types-of-referential-actions"
},
{
"source": "/d/postgres-set-null",
"destination": "https://www.prisma.io/docs/concepts/components/prisma-schema/relations/referential-actions#special-cases-for-referential-actions"
},
{
"source": "/d/mysql-set-default",
"destination": "https://www.prisma.io/docs/concepts/components/prisma-schema/relations/referential-actions#special-cases-for-referential-actions"
},

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This mirrors the preview link mentioned in: prisma/docs#4132 (comment)

@janpio
Copy link
Member

janpio commented Nov 28, 2022

Let's make sure the links at least exist (and can not be forgotten), and then follow up via another PR.

@janpio janpio merged commit afd5477 into main Nov 28, 2022
@janpio janpio deleted the feat/add-referential-actions-odd-cases-links branch November 28, 2022 08:12
@janpio
Copy link
Member

janpio commented Nov 28, 2022

And #80

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants