-
Notifications
You must be signed in to change notification settings - Fork 2.8k
console: modify default value for PK fix #4075 #4679
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
console: modify default value for PK fix #4075 #4679
Conversation
|
Beep boop! 🤖 Hey @fkhadra, thanks for your PR! One of my human friends will review this PR and get back to you as soon as possible. Stay awesome! 😎 |
|
Review app for commit 2f2386c deployed to Heroku: https://hge-ci-pull-4679.herokuapp.com |
beerose
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your PR!
Code looks good, approving.
|
Deploy preview for hasura-docs ready! Built with commit 9d01094 |
|
Review app for commit c8a0a30 deployed to Heroku: https://hge-ci-pull-4679.herokuapp.com |
group console notes together Co-authored-by: Aleksandra Sikora <ola.zxcvbnm@gmail.com>
|
Updated :) |
|
Review app for commit 5aadfe9 deployed to Heroku: https://hge-ci-pull-4679.herokuapp.com |
|
Review app for commit e336239 deployed to Heroku: https://hge-ci-pull-4679.herokuapp.com |
|
Review app for commit 9d01094 deployed to Heroku: https://hge-ci-pull-4679.herokuapp.com |
|
Review app https://hge-ci-pull-4679.herokuapp.com is deleted |
|
Beep boop! 🤖 Awesome work @fkhadra! All of us at Hasura ❤️ what you did. Thanks again 🤗 |
Description
Hey,
This PR allows changing the default value for a Primary key when editing the column. I also added a cypress test for that.
Changelog
CHANGELOG.mdis updated with user-facing content relevant to this PR.Affected components
Related Issues
Related issue #4075
Catalog upgrade
Does this PR change Hasura Catalog version?
Metadata
Does this PR add a new Metadata feature?
GraphQL
Breaking changes