-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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: allow user to cascade Postgres dependencies when dropping Postgres objects (close #5109) #5248
Conversation
Deploy preview for hasura-docs ready! Built with commit 3ed8d1e |
Review app for commit 2827931 deployed to Heroku: https://hge-ci-pull-5248.herokuapp.com |
Co-authored-by: Aleksandra Sikora <aleksandra@hasura.io>
…o feat/pg_cascade_prompt
…nkar/graphql-engine into feat/pg_cascade_prompt
Review app for commit 30ded79 deployed to Heroku: https://hge-ci-pull-5248.herokuapp.com |
Review app for commit 67bd1c1 deployed to Heroku: https://hge-ci-pull-5248.herokuapp.com |
Review app for commit 6dc6490 deployed to Heroku: https://hge-ci-pull-5248.herokuapp.com |
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.
UX approved
Review app for commit 6ec474f deployed to Heroku: https://hge-ci-pull-5248.herokuapp.com |
Review app for commit 7ff057e deployed to Heroku: https://hge-ci-pull-5248.herokuapp.com |
CHANGELOG.md
Outdated
@@ -19,6 +19,7 @@ This release contains the [PDV refactor (#4111)](https://github.com/hasura/graph | |||
- server: some mutations that cannot be performed will no longer be in the schema (for instance, `delete_by_pk` mutations won't be shown to users that do not have select permissions on all primary keys) (#4111) | |||
- server: miscellaneous description changes (#4111) | |||
- server: treat the absence of `backend_only` configuration and `backend_only: false` equally (closing #5059) (#4111) | |||
- console: add user prompt to cascade delete on Postgres dependency errors (close #5109) (#5248) |
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.
allow user to cascade Postgres dependencies when dropping Postgres objects
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.
changelog
Review app for commit d8f5ad8 deployed to Heroku: https://hge-ci-pull-5248.herokuapp.com |
Review app https://hge-ci-pull-5248.herokuapp.com is deleted |
…stgres objects (close hasura#5109) (hasura#5248)
@@ -689,6 +693,7 @@ const makeMigrationCall = ( | |||
successMsg, | |||
errorMsg, | |||
shouldSkipSchemaReload, | |||
false, |
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.
Description
This PR adds a user prompt when the user gets postgress error
dependent_objects_still_exist-2BP01
Changelog
CHANGELOG.md
is updated with user-facing content relevant to this PR. If no changelog is required, then add theno-changelog-required
label.Affected components
Related Issues
closes #5109
Solution and Design
This PR will add an on error listener at
makeMigrationCall
which makes sure thatSteps to test and verify
Limitations, known bugs & workarounds
Does this PR add a new Metadata feature?
GraphQL
Breaking changes