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

Add permission to groups of users access the event_streams_view #17723

Merged
merged 1 commit into from
Aug 3, 2018

Conversation

CharlleDaniel
Copy link
Member

This PR is able to:

  • Add permission to access the event_streams_view in the follow groups:
    • administrator
    • desktop
    • operator
    • security
    • support
    • user
    • reader

@miq-bot
Copy link
Member

miq-bot commented Jul 18, 2018

Checked commit CharlleDaniel@c1b83fa with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 🏆

@CharlleDaniel
Copy link
Member Author

@miq-bot add_reviewer @agrare

@miq-bot miq-bot requested a review from agrare July 25, 2018 02:12
@miq-bot

This comment has been minimized.

@CharlleDaniel
Copy link
Member Author

@miq-bot assign @agrare

@agrare
Copy link
Member

agrare commented Jul 26, 2018

@martinpovolny @h-kataria can you review?

@agrare agrare assigned h-kataria and unassigned agrare Aug 1, 2018
@h-kataria
Copy link
Contributor

looks good. @CharlleDaniel is this an enhancement or a bug fix, please add labels appropriately.

@miq-bot

This comment has been minimized.

@miq-bot

This comment has been minimized.

@CharlleDaniel
Copy link
Member Author

@miq-bot add_label enhancement

@CharlleDaniel
Copy link
Member Author

@miq-bot add_label api

@miq-bot miq-bot added the api label Aug 3, 2018
@CharlleDaniel
Copy link
Member Author

@h-kataria done, could you merge now?

@h-kataria h-kataria added this to the Sprint 92 Ending Aug 13, 2018 milestone Aug 3, 2018
@h-kataria h-kataria merged commit ce1baa2 into ManageIQ:master Aug 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants