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
Fix bulk deleting roles without permissions close #4178 #4195
Conversation
Beep boop! 🤖 Hey @sobrinho, 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 a551c02 deployed to Heroku: https://hge-ci-pull-4195.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.
Changelog approved.
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.
In place of completely hiding the bulk select checkbox if no permissions exist, it would be better to simply disable the check box and changing its title to "No permissions exist" to make it clear why it cannot be selected. Adding a new key called disableCheckbox
along with the showCheckbox
key should help achieve this
Alright, I will do it. |
Deploy preview for hasura-docs ready! Built with commit 9183a19 |
@rikinsk done! |
Review app for commit c485d6d deployed to Heroku: https://hge-ci-pull-4195.herokuapp.com |
@beerose If you have a role used in one table but not used in another table |
Thank you! Approving. |
Deploy preview for hasura-docs ready! Built with commit cb5d94a |
Review app for commit cb5d94a deployed to Heroku: https://hge-ci-pull-4195.herokuapp.com |
Review app for commit c466844 deployed to Heroku: https://hge-ci-pull-4195.herokuapp.com |
Review app https://hge-ci-pull-4195.herokuapp.com is deleted |
Beep boop! 🤖 Awesome work @sobrinho! All of us at Hasura ❤️ what you did. Thanks again 🤗 |
Description
Roles without any permissions can't be bulk deleted and throws an error as described in #4178.
Changelog
CHANGELOG.md
is updated with user-facing content relevant to this PR.Affected components
Related Issues
#4178
Solution and Design
Do not render the checkbox for roles without any permission. That way, the user can't select it.
Steps to test and verify
Try to mark and bulk delete a role without any permissions configured in a table, watch the inspector before doing it.
Limitations, known bugs & workarounds
N/A
Server checklist
N/A
Catalog upgrade
Does this PR change Hasura Catalog version?
Metadata
Does this PR add a new Metadata feature?
run_sql
auto manages the new metadata through schema diffing?run_sql
auto manages the definitions of metadata on renaming?export_metadata
/replace_metadata
supports the new metadata added?GraphQL
Breaking changes
No Breaking changes
There are breaking changes:
Metadata API
Existing
query
types:args
payload which is not backward compatibleJSON
schemaGraphQL API
Schema Generation:
NamedType
Schema Resolve:-
null
value for any input fieldsLogging
JSON
schema has changedtype
names have changed