Bug 1348796 - No gtl view buttons in cloud key pair page
Summary: No gtl view buttons in cloud key pair page
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.6.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.7.0
Assignee: Eric Hayes
QA Contact: Aziza Karol
URL:
Whiteboard: view:ui
Depends On:
Blocks: 1353280
TreeView+ depends on / blocked
 
Reported: 2016-06-22 06:17 UTC by Aziza Karol
Modified: 2017-01-04 12:56 UTC (History)
7 users (show)

Fixed In Version: 5.7.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1353280 (view as bug list)
Environment:
Last Closed: 2017-01-04 12:56:01 UTC
Category: Bug
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
snapshot (42.74 KB, image/png)
2016-06-22 06:17 UTC, Aziza Karol
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2017:0012 0 normal SHIPPED_LIVE CFME 5.7.0 bug fixes and enhancement update 2017-01-04 17:50:36 UTC

Description Aziza Karol 2016-06-22 06:17:02 UTC
Created attachment 1170598 [details]
snapshot

Description of problem:


Version-Release number of selected component (if applicable):
5.6.0.10

How reproducible:
100%

Steps to Reproduce:
In Default view settings there is an option to set gtl view for key pairs, but in 
cloud key pairs page gtl view buttons are missing.
see attached screenshot

Actual results:


Expected results:


Additional info:

Comment 1 Harpreet Kataria 2016-06-22 13:37:04 UTC
Eric,

Looks like we need to add @layout for Cloud Key Pairs screen to the list here https://github.com/ManageIQ/manageiq/blob/master/app/helpers/application_helper.rb#L1060

Let me know if you have questions.

~Harpreet

Comment 4 CFME Bot 2016-06-23 20:06:44 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/e3db9d1d079d2849f25a742d0e37a3622df2ea36

commit e3db9d1d079d2849f25a742d0e37a3622df2ea36
Author:     Eric Hayes <eric>
AuthorDate: Wed Jun 22 12:13:21 2016 -0700
Commit:     Eric Hayes <eric>
CommitDate: Thu Jun 23 07:59:26 2016 -0700

    Allow gtl view for Cloud Key Pairs
    
    * Adds `auth_key_pair_cloud` to layout configuration constant
    * Adds kludge in `quadicon_helper` for detecting Authentication class and setting correct controller. (I’m working on refactoring this.)
    * Cleans up some style and adds a note for future refactoring
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1348796

 app/helpers/application_helper.rb     | 18 +++++++++---------
 app/helpers/quadicon_helper.rb        |  3 +++
 app/models/authentication.rb          |  5 +++++
 app/views/layouts/gtl/_list.html.haml |  2 ++
 4 files changed, 19 insertions(+), 9 deletions(-)

Comment 6 CFME Bot 2016-07-06 16:51:24 UTC
New commit detected on ManageIQ/manageiq/darga:
https://github.com/ManageIQ/manageiq/commit/0e179d7f9e0ce9b0cb5ed29326978f43f81ac4a4

commit 0e179d7f9e0ce9b0cb5ed29326978f43f81ac4a4
Author:     Eric Hayes <eric>
AuthorDate: Tue Jul 5 11:50:27 2016 -0700
Commit:     Eric Hayes <eric>
CommitDate: Tue Jul 5 13:29:11 2016 -0700

    Allow gtl view for Cloud Key Pairs
    
    * Adds `auth_key_pair_cloud` to layout configuration constant
    * Adds kludge in `quadicon_helper` for detecting Authentication class and setting correct controller. (I’m working on refactoring this.)
    * Cleans up some style and adds a note for future refactoring
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1348796
    (cherry picked from commit e3db9d1d079d2849f25a742d0e37a3622df2ea36)

 app/helpers/application_helper.rb     | 18 +++++++++---------
 app/helpers/quadicon_helper.rb        |  3 +++
 app/models/authentication.rb          |  5 +++++
 app/views/layouts/gtl/_list.html.haml |  1 +
 4 files changed, 18 insertions(+), 9 deletions(-)

Comment 11 Aziza Karol 2016-11-04 05:12:38 UTC
GTL view buttons added in cloud key pair page.

Verified:5.7.0.9-beta2.1

Comment 12 Aziza Karol 2016-11-04 05:14:09 UTC
However, updating the gtl view settings does not work.I have logged a separate bz https://bugzilla.redhat.com/show_bug.cgi?id=1391792 to track this issue

Comment 14 errata-xmlrpc 2017-01-04 12:56:01 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2017-0012.html


Note You need to log in before you can comment on or make changes to this bug.