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.

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

Update 125-creating-the-prisma-schema.mdx #2960

Merged
merged 1 commit into from
Mar 31, 2022
Merged

Conversation

Nditah
Copy link
Contributor

@Nditah Nditah commented Mar 9, 2022

Fix: Update Post Schema: Change user to author and userId to authorId

Describe this PR

Changes

What issue does this fix?

Any other relevant information

Fix: Update Post Schema:   Change user to author and userId to authorId
@netlify
Copy link

netlify bot commented Mar 9, 2022

✔️ Deploy Preview for prisma2-docs ready!
Built without sensitive environment variables

🔨 Explore the source changes: 2c7e5b3

🔍 Inspect the deploy log: https://app.netlify.com/sites/prisma2-docs/deploys/6228a2cd77bb4f0008ec45a5

😎 Browse the preview: https://deploy-preview-2960--prisma2-docs.netlify.app

@janpio
Copy link
Contributor

janpio commented Mar 9, 2022

Why this change? Is the current version broken? Is this "just" a question of style?

@Nditah
Copy link
Contributor Author

Nditah commented Mar 9, 2022 via email

@tanberry
Copy link
Contributor

Thanks @Nditah for raising this question.... I'm not sure that changing user to author is the best way to demonstrate the relation between those two. This relation is explained here in our docs....

@matthewmueller can you take a look at this and share your opinion, please?

@tanberry
Copy link
Contributor

Let's go ahead and merge this, thanks @Nditah , and I'll open a new Issue for going through that whole set of docs and making them more consistent about userID versus authorID.

@tanberry tanberry merged commit b3b60d3 into prisma:main Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants