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

Refactor EmsCommon nested lists using GenericShowMixin #427

Merged
merged 11 commits into from
Feb 24, 2017

Conversation

martinpovolny
Copy link
Member

No description provided.

@martinpovolny martinpovolny force-pushed the ems_common_generic_show branch from f17f8ef to 1ee9bab Compare February 18, 2017 21:11
@martinpovolny martinpovolny changed the title Refactor EmsCommon nestes lists using GenericShowMixin Refactor EmsCommon nested lists using GenericShowMixin Feb 18, 2017
@martinpovolny martinpovolny force-pushed the ems_common_generic_show branch from 1ee9bab to a3b8c9c Compare February 22, 2017 20:30
@martinpovolny martinpovolny force-pushed the ems_common_generic_show branch from 3c1c249 to 62d54de Compare February 23, 2017 13:06
@mzazrivec mzazrivec self-assigned this Feb 23, 2017
@martinpovolny martinpovolny force-pushed the ems_common_generic_show branch from 62d54de to 48830b6 Compare February 24, 2017 08:20
@martinpovolny martinpovolny force-pushed the ems_common_generic_show branch from 4e8b766 to 1c4232b Compare February 24, 2017 09:00
@miq-bot
Copy link
Member

miq-bot commented Feb 24, 2017

Checked commits martinpovolny/manageiq-ui-classic@a07b8bc~...1c4232b with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
5 files checked, 2 offenses detected

app/controllers/ems_common.rb

spec/controllers/ems_infra_controller_spec.rb

@mzazrivec mzazrivec added this to the Sprint 55 Ending Feb 27, 2017 milestone Feb 24, 2017
@mzazrivec mzazrivec merged commit 9c9f774 into ManageIQ:master Feb 24, 2017
@martinpovolny martinpovolny deleted the ems_common_generic_show branch March 8, 2017 08:44
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