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

Catch constraint errors on role delete #1344

Merged
merged 1 commit into from
Feb 10, 2017

Conversation

vanstee
Copy link
Member

@vanstee vanstee commented Feb 6, 2017

If trying to delete a role via the API, the server will response with a 422 and the body: {"errors":{"id":["cannot delete role that has been granted permissions"]}} instead of throwing an exception.

Fixes #1336

@vanstee vanstee added the review label Feb 6, 2017
@vanstee vanstee merged commit 3b801e0 into master Feb 10, 2017
@vanstee vanstee deleted the vanstee/role-delete-constraints branch February 10, 2017 20:41
@vanstee vanstee removed the review label Feb 10, 2017
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