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

feat(prisma/migrations): Default value on new column #465

Merged
merged 17 commits into from Oct 28, 2023
Merged

Conversation

Natoandro
Copy link
Contributor

@Natoandro Natoandro commented Oct 24, 2023

  • Display a more concise error message for new column that failed the NON NULL constraint during the migration.
  • Enable user to set default value on new NON NULL column.

@Natoandro Natoandro force-pushed the migration-recovery branch 2 times, most recently from 0d0a5fb to 36d103b Compare October 24, 2023 21:44
typegate/deno.json Outdated Show resolved Hide resolved
@Natoandro Natoandro marked this pull request as ready for review October 25, 2023 15:51
Signed-off-by: Natoandro <anatoandro@hotmail.com>
libs/common/src/typegraph/runtimes/prisma.rs Show resolved Hide resolved
typegate/tests/utils/test_module.ts Outdated Show resolved Hide resolved
typegate/tests/e2e/cli/deploy_test.ts Outdated Show resolved Hide resolved
@Natoandro Natoandro requested a review from zifeo October 26, 2023 15:26
Signed-off-by: Teo Stocco <zifeo@users.noreply.github.com>
@zifeo zifeo merged commit 0702818 into main Oct 28, 2023
10 of 11 checks passed
@zifeo zifeo deleted the migration-recovery branch October 28, 2023 14:24
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.

None yet

2 participants