-
Notifications
You must be signed in to change notification settings - Fork 357
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
Added changes to show Catalog Item type #165
Added changes to show Catalog Item type #165
Conversation
c17d160
to
8b8b149
Compare
@@ -111,7 +111,7 @@ def atomic_st_edit | |||
@edit[:new] ||= {} | |||
@edit[:current] ||= {} | |||
@edit[:key] = "prov_edit__new" | |||
@edit[:st_prov_types] = catalog_item_types |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should not be putting constants in the session. I cannot imagine a reason to do this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@martinpovolny moved constant out of session.
Looks good except for the in-line comment above. |
Show nice display value for Catalog Item type in list view, Catalog Item detail view and as read only text on the add/edit screens. Moved list of catalog item types out of @edit hash, we can directly reference the constant from views. https://bugzilla.redhat.com/show_bug.cgi?id=1348239
8b8b149
to
faf9999
Compare
Checked commit h-kataria@faf9999 with ruby 2.2.6, rubocop 0.46.0, and haml-lint 0.19.0 |
Show Catalog ITem type of list view, Catalog Item detail view and as read only text on the add/edit screens
https://bugzilla.redhat.com/show_bug.cgi?id=1348239
@dclarizio please review, this PR depends on changes in core repo in ManageIQ/manageiq#13516
UI screenshots