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

Restore resolvedData types #7833

Merged
merged 4 commits into from
Aug 24, 2022
Merged

Conversation

Noviny
Copy link
Contributor

@Noviny Noviny commented Aug 23, 2022

The resolvedData types were identified as being actively misleading in #7801.

This pull request restores the types, and now matches what Keystone actually expects and provides to end developers.

The new types approximately mirror the prisma types, but are slightly different for database fields with kind: 'multi' (multi-column fields) that Keystone and different field types often use.

@changeset-bot
Copy link

changeset-bot bot commented Aug 23, 2022

🦋 Changeset detected

Latest commit: 153698d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@keystone-6/core Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Aug 23, 2022

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

Name Status Preview Updated
keystone-next-docs ✅ Ready (Inspect) Visit Preview Aug 24, 2022 at 4:31AM (UTC)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 23, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 153698d:

Sandbox Source
@keystone-6/sandbox Configuration

@vercel vercel bot temporarily deployed to Preview August 23, 2022 05:31 Inactive
@dcousens dcousens changed the title improve resolvedData types Restore resolvedData types Aug 23, 2022
@vercel vercel bot temporarily deployed to Preview August 24, 2022 00:02 Inactive
@vercel vercel bot temporarily deployed to Preview August 24, 2022 02:46 Inactive
@dcousens dcousens force-pushed the improve-types-for-prisma-create-update branch from f0b488e to 926bc79 Compare August 24, 2022 03:17
@vercel vercel bot temporarily deployed to Preview August 24, 2022 03:19 Inactive
@dcousens dcousens force-pushed the improve-types-for-prisma-create-update branch from 926bc79 to 19cd173 Compare August 24, 2022 03:27
@vercel vercel bot temporarily deployed to Preview August 24, 2022 03:29 Inactive
@dcousens dcousens force-pushed the improve-types-for-prisma-create-update branch from 19cd173 to e24c535 Compare August 24, 2022 03:37
@vercel vercel bot temporarily deployed to Preview August 24, 2022 03:40 Inactive
@dcousens dcousens force-pushed the improve-types-for-prisma-create-update branch 2 times, most recently from 4865517 to a57d96d Compare August 24, 2022 04:07
@vercel vercel bot temporarily deployed to Preview August 24, 2022 04:11 Inactive
@dcousens dcousens enabled auto-merge (squash) August 24, 2022 04:13
@vercel vercel bot temporarily deployed to Preview August 24, 2022 04:14 Inactive
@dcousens dcousens force-pushed the improve-types-for-prisma-create-update branch from 7b49a2f to 153698d Compare August 24, 2022 04:26
@vercel vercel bot temporarily deployed to Preview August 24, 2022 04:31 Inactive
@dcousens dcousens merged commit fdbfca3 into main Aug 24, 2022
@dcousens dcousens deleted the improve-types-for-prisma-create-update branch August 24, 2022 04:37
dcousens added a commit that referenced this pull request Sep 14, 2022
Co-authored-by: Daniel Cousens <dcousens@users.noreply.github.com>
dcousens added a commit that referenced this pull request Sep 15, 2022
Co-authored-by: Daniel Cousens <dcousens@users.noreply.github.com>
dcousens added a commit that referenced this pull request Sep 15, 2022
Co-authored-by: Daniel Cousens <dcousens@users.noreply.github.com>
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.

Type of resolvedData in hooks should use Prisma create/update types
2 participants