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

[WIP] Expose Custom Buttons for Generic Objects in Service #2554

Closed

Conversation

lgalis
Copy link
Contributor

@lgalis lgalis commented Oct 30, 2017

  • Display the Generic object definitions and classes from within the service controller
  • Display associated custom buttons for generic objects in the service controller

Links

https://www.pivotaltracker.com/story/show/150933586

screenshot from 2017-11-03 17-03-49
screenshot from 2017-11-03 17-03-59
screenshot from 2017-11-03 18-00-22

@ohadlevy
Copy link
Member

@igalis 1. please rebase, 2. please add tests...

@lgalis
Copy link
Contributor Author

lgalis commented Oct 30, 2017

@ohadlevy - this is WIP - not ready yet

@miq-bot miq-bot added the wip label Oct 30, 2017
@miq-bot
Copy link
Member

miq-bot commented Oct 30, 2017

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

@lgalis lgalis force-pushed the expose_custom_buttons_for_generic_objects branch from a6c5805 to 6137d23 Compare October 30, 2017 13:32
@lgalis lgalis force-pushed the expose_custom_buttons_for_generic_objects branch 3 times, most recently from 2607233 to a7e2c5d Compare October 30, 2017 23:57
@miq-bot
Copy link
Member

miq-bot commented Nov 3, 2017

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

@lgalis lgalis force-pushed the expose_custom_buttons_for_generic_objects branch from a7e2c5d to bc6ba0c Compare November 3, 2017 21:03
@lgalis lgalis force-pushed the expose_custom_buttons_for_generic_objects branch from bc6ba0c to 8d45aa8 Compare November 3, 2017 22:00
@lgalis lgalis force-pushed the expose_custom_buttons_for_generic_objects branch from 8d45aa8 to 2dc80d2 Compare November 4, 2017 22:11
@miq-bot
Copy link
Member

miq-bot commented Nov 4, 2017

Checked commits lgalis/manageiq-ui-classic@8f8d326~...2dc80d2 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
8 files checked, 6 offenses detected

app/controllers/service_controller.rb

app/helpers/application_helper/toolbar_builder.rb

@lgalis
Copy link
Contributor Author

lgalis commented Nov 7, 2017

Close in favor of #2636

@lgalis lgalis closed this Nov 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants