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

Define model_name with route keys for the StorageManager model #17513

Merged
merged 1 commit into from
Jun 4, 2018

Conversation

skateman
Copy link
Member

@skateman skateman commented Jun 4, 2018

I am not sure if this is something we want, but it would make the UI happier and it looks consistent. Read more about this here and here. Also tightly related to #17512.

@miq-bot
Copy link
Member

miq-bot commented Jun 4, 2018

Checked commit skateman@781f7c9 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. 🍪

Copy link
Member

@agrare agrare left a comment

Choose a reason for hiding this comment

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

LGTM

@agrare agrare self-assigned this Jun 4, 2018
@agrare agrare merged commit 086c465 into ManageIQ:master Jun 4, 2018
@agrare agrare added this to the Sprint 87 Ending Jun 4, 2018 milestone Jun 4, 2018
@skateman skateman deleted the storage-routes branch June 4, 2018 15:02
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