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 chargeback to shortcuts to allow access to chargeback only. #14809

Merged
merged 1 commit into from
Apr 19, 2017

Conversation

martinpovolny
Copy link
Member

https://bugzilla.redhat.com/show_bug.cgi?id=1343418

Create a role, group and then a user that has only 'chargeback' feature enabled.

Try to login (you get an error message). Expected result: you should login and see only Chargeback stuff.

To test this you might need to run MiqShortcut.seed to force reseeding the MiqShortcuts.

@martinpovolny
Copy link
Member Author

Not adding the flag, but this can be backported to Fine.

@miq-bot
Copy link
Member

miq-bot commented Apr 19, 2017

Checked commit martinpovolny@0a5ae7c with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks good. 👍

Copy link
Member

@isimluk isimluk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@isimluk isimluk self-assigned this Apr 19, 2017
@isimluk isimluk merged commit b3a9c9a into ManageIQ:master Apr 19, 2017
simaishi pushed a commit that referenced this pull request Apr 20, 2017
Add chargeback to shortcuts to allow access to chargeback only.
(cherry picked from commit b3a9c9a)
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit 525ecae84eed77c85d93944b148d85247ba94159
Author: Šimon Lukašík <[email protected]>
Date:   Wed Apr 19 16:48:18 2017 +0200

    Merge pull request #14809 from martinpovolny/chargeback_shortcut
    
    Add chargeback to shortcuts to allow access to chargeback only.
    (cherry picked from commit b3a9c9a63c8b66ae672e1e1a16fdf298dc8d86b2)

@isimluk isimluk added this to the Sprint 59 Ending Apr 24, 2017 milestone Apr 20, 2017
@martinpovolny martinpovolny deleted the chargeback_shortcut branch November 28, 2017 18:54
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