Bug 1740408 - Breadcrumbs incorrect in Chargeback/Rates and Chargeback/Assignments
Summary: Breadcrumbs incorrect in Chargeback/Rates and Chargeback/Assignments
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.11.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.11.5
Assignee: Richard Vsiansky
QA Contact: Parthvi Vala
Red Hat CloudForms Documentation
URL:
Whiteboard: testathon
Depends On:
Blocks: 1740746
TreeView+ depends on / blocked
 
Reported: 2019-08-12 21:02 UTC by ecrosby
Modified: 2020-05-05 13:43 UTC (History)
9 users (show)

Fixed In Version: 5.11.5.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-05-05 13:43:06 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:
pm-rhel: cfme-5.11.z+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2020:2020 0 None None None 2020-05-05 13:43:24 UTC

Description ecrosby 2019-08-12 21:02:53 UTC
Description of problem:
When browsing to Chargeback/Rates or Chargeback/Assignments, Rates and Assignments are listed twice in the breadcrumbs. Also, chargeback is listed with an "s"

Version-Release number of selected component (if applicable):
cfme-5.11.0.18-1.el8cf.x86_64

How reproducible:
Consistently

Steps to Reproduce:
Navigate to Overview > Chargeback > Rates (or assignments) > Compute
1. Breadcrumbs read strangely as Rates is repeated in the string ("Chargeback > Rates > Rates > Compute").
2. Breadcrumbs -- Chargebacks shouldn't have an "s".

Comment 2 Richard Vsiansky 2019-08-14 13:01:11 UTC
Hi, 

the typo will be fixed, however the double breadcrumbs are correct. The first breadcrumb is for accordion header and the second one is for the root of the tree. I know that it does not look good when they are the same, but breadcrumbs just mirror the structure of the application and in this case the structure is not alright. (We should not have accordion with only one tree.. at least it doesn't make sense for me. But this is a subject for a different conversation.)

Richard

Comment 4 CFME Bot 2019-08-14 14:36:48 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/7ab5f537ca3e0c1093a057b6f7a86001c3894d9b
commit 7ab5f537ca3e0c1093a057b6f7a86001c3894d9b
Author:     Richard Vsiansky <r.vsia>
AuthorDate: Wed Aug 14 08:54:13 2019 -0400
Commit:     Richard Vsiansky <r.vsia>
CommitDate: Wed Aug 14 08:54:13 2019 -0400

    Fix typo in chargeback breadcrumbs options

    https://bugzilla.redhat.com/show_bug.cgi?id=1740408

 app/controllers/chargeback_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 5 CFME Bot 2020-03-20 17:10:44 UTC
New commit detected on ManageIQ/manageiq-ui-classic/ivanchuk:

https://github.com/ManageIQ/manageiq-ui-classic/commit/39d9a5e0c87fc502edbd8890a9fb87879c815768
commit 39d9a5e0c87fc502edbd8890a9fb87879c815768
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Wed Aug 14 14:35:06 2019 +0000
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Wed Aug 14 14:35:06 2019 +0000

    Merge pull request #6024 from rvsia/fix-charheback-typo

    Fix typo in chargeback breadcrumbs options

    (cherry picked from commit b60e11d97b76ca0c8890b829228ee45703b7c99b)

    https://bugzilla.redhat.com/show_bug.cgi?id=1740408

 app/controllers/chargeback_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 6 Parthvi Vala 2020-04-16 12:13:41 UTC
FIXED. Verified on 5.11.5.1.

Comment 9 errata-xmlrpc 2020-05-05 13:43:06 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://access.redhat.com/errata/RHBA-2020:2020


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