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 plugin display name #135

Merged
merged 1 commit into from
Aug 30, 2018
Merged

Conversation

mzazrivec
Copy link
Contributor

See ManageIQ/manageiq-ui-classic#4496 for primary motivation.

@miha-plesko
Copy link
Contributor

Looks good, should we merge or wait the UI PR to get in first?

@mzazrivec
Copy link
Contributor Author

Probably better to wait for the UI PR.

@miq-bot
Copy link
Member

miq-bot commented Aug 23, 2018

Checked commit mzazrivec@a1a5c23 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

@miha-plesko miha-plesko merged commit 9a38df8 into ManageIQ:master Aug 30, 2018
@miha-plesko miha-plesko self-assigned this Aug 30, 2018
@miha-plesko miha-plesko added this to the Sprint 94 Ending Sept 10, 2018 milestone Aug 30, 2018
@mzazrivec mzazrivec deleted the add_plugin_name branch August 30, 2018 12:10
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.

3 participants