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

Adding roles, filters and configurations for Physical Switches support #17216

Merged
merged 1 commit into from
May 3, 2018

Conversation

saulotoledo
Copy link
Member

@saulotoledo saulotoledo commented Mar 28, 2018

This PR adds roles, filters and configurations for Physical Switches support in the UI.

Depends on:

@saulotoledo saulotoledo changed the title Adding roles, filters and configurations for Physical Switches support [DEPENDS ON #16948] Adding roles, filters and configurations for Physical Switches support Mar 28, 2018
@saulotoledo saulotoledo changed the title [DEPENDS ON #16948] Adding roles, filters and configurations for Physical Switches support [DEPENDS ON: 16948] Adding roles, filters and configurations for Physical Switches support Mar 28, 2018
@saulotoledo
Copy link
Member Author

@agrare, @rodneyhbrown7, this can be reviewed after merging #16948.

@miq-bot
Copy link
Member

miq-bot commented Apr 4, 2018

This pull request is not mergeable. Please rebase and repush.

@saulotoledo saulotoledo force-pushed the add_switches_config branch 2 times, most recently from a738550 to 9b8f07f Compare April 5, 2018 15:48
@saulotoledo saulotoledo force-pushed the add_switches_config branch 5 times, most recently from bf0ce3b to cdaa8d8 Compare April 6, 2018 12:59
@saulotoledo saulotoledo changed the title [DEPENDS ON: 16948] Adding roles, filters and configurations for Physical Switches support Adding roles, filters and configurations for Physical Switches support Apr 6, 2018
@saulotoledo saulotoledo force-pushed the add_switches_config branch 2 times, most recently from ba57225 to 4f9c1de Compare April 12, 2018 15:17
@saulotoledo
Copy link
Member Author

@agrare, could you take a look on this PR?

@miq-bot
Copy link
Member

miq-bot commented Apr 23, 2018

Checked commit saulotoledo@d278d6b 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. 🍪

@agrare agrare self-assigned this Apr 23, 2018
@agrare
Copy link
Member

agrare commented Apr 23, 2018

Not sure where to start on this one but I think @martinpovolny is the one to look at user/roles changes

@agrare agrare assigned martinpovolny and unassigned agrare May 2, 2018
@martinpovolny
Copy link
Member

To test (to see the changes in the feature tree) this PR has to be tested together with ManageIQ/manageiq-ui-classic#3699

Having done so I see no problems --> merging.

@martinpovolny martinpovolny merged commit 7269dcc into ManageIQ:master May 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