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

Fix permission handling for editing/deleting roles #4270

Merged
merged 1 commit into from Oct 19, 2017
Merged

Fix permission handling for editing/deleting roles #4270

merged 1 commit into from Oct 19, 2017

Conversation

@kroepke
Copy link
Member

@kroepke kroepke commented Oct 19, 2017

  • Fix permission checks in RoleResource
  • Conditionally display buttons in RoleList component

Fixes #4254
Refs #4265
(cherry picked from commit 5c38d3a)

* Fix permission checks in RoleResource

* Conditionally display buttons in RoleList component

(cherry picked from commit 5c38d3a)
@kroepke kroepke added this to the 2.3.2 milestone Oct 19, 2017
@ghost ghost assigned kroepke Oct 19, 2017
@joschi
joschi approved these changes Oct 19, 2017
@joschi joschi changed the title Fix permission handling for editing/deleting roles (#4265) Fix permission handling for editing/deleting roles Oct 19, 2017
@joschi joschi merged commit 5917767 into 2.3 Oct 19, 2017
3 of 5 checks passed
3 of 5 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
@garybot2
ci-web-linter Jenkins build graylog-pr-linter-check 2020 has succeeded
Details
@garybot2
graylog-project/pr Jenkins build graylog-project-pr-snapshot 631 has succeeded
Details
license/cla Contributor License Agreement is signed.
Details
@joschi joschi deleted the backport-4254 branch Oct 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants