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 features to physical servers pages #14709

Merged
merged 2 commits into from
Apr 19, 2017
Merged

Add features to physical servers pages #14709

merged 2 commits into from
Apr 19, 2017

Conversation

gabrielsvinha
Copy link
Contributor

@gabrielsvinha gabrielsvinha commented Apr 10, 2017

  • Adds operations for physical server buttons.
  • Adds modify actions for physical server buttons.

@rodneyhbrown7
Copy link

Gabriel this overlaps with #14028

@@ -6399,8 +6399,54 @@
:identifier: physical_server_show_list
- :name: Show
:description: Display Individual Physical Server
:feature_type: view
:feature_type: view
Copy link
Member

Choose a reason for hiding this comment

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

please, remove the extra space here

thx!

Copy link
Contributor Author

@gabrielsvinha gabrielsvinha Apr 19, 2017

Choose a reason for hiding this comment

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

Done.

@martinpovolny
Copy link
Member

martinpovolny commented Apr 19, 2017

@gabrielsvinha : does this really depend on the UI PR? I guess it's the other way round, isn't it?We need this one to merge first, right?

I think that we can merge this except for the extra space.

@rodneyhbrown7 : do you agree or should #14028 be merged first?

@gabrielsvinha
Copy link
Contributor Author

@martinpovolny you're right. UI depends on this one, thnks.

@miq-bot
Copy link
Member

miq-bot commented Apr 19, 2017

Checked commits https://github.com/lenovo/manageiq/compare/671db9a91da149a78a6bf908892ee0e5a6131d0d~...d28394d2d5919c43080bb9efa9170d08f64edf65 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks good. 👍

Removes spaces from miq product features
@rodneyhbrown7
Copy link

Agreed this one can be merged first.

@martinpovolny martinpovolny merged commit a7854a2 into ManageIQ:master Apr 19, 2017
@martinpovolny martinpovolny added this to the Sprint 59 Ending Apr 24, 2017 milestone Apr 19, 2017
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.

5 participants