Project

General

Profile

Bug #23559

Breadcrumbs switcher on trends page doesn't work for predefined trend types

Added by Tomáš Strachota 4 months ago. Updated 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Predefined trends ( = other than fact trends) have hardcoded names and API returns name = null. That results in empty lines in the breadcrumb switcher and several js warnings when the switcher is opened:

warning.js:33 Warning: Failed prop type: The prop `resourceSwitcherItems[0].name` is marked as required in `BreadcrumbBar`, but its value is `null`.
    in BreadcrumbBar (created by Connect(BreadcrumbBar))
    in Connect(BreadcrumbBar)
    in Provider
warning.js:33 Warning: Failed prop type: The prop `resources[0].name` is marked as required in `BreadcrumbSwitcher`, but its value is `null`.
    in BreadcrumbSwitcher (created by BreadcrumbBar)
    in BreadcrumbBar (created by Connect(BreadcrumbBar))
    in Connect(BreadcrumbBar)
    in Provider
warning.js:33 Warning: Failed prop type: The prop `resources[0].name` is marked as required in `BreadcrumbSwitcherPopover`, but its value is `null`.
    in BreadcrumbSwitcherPopover (created by BreadcrumbSwitcher)
    in BreadcrumbSwitcher (created by BreadcrumbBar)
    in ol (created by Breadcrumb)
    in Breadcrumb (created by Breadcrumb)
    in Breadcrumb (created by Breadcrumb)
    in Breadcrumb (created by BreadcrumbBar)
    in div (created by BreadcrumbBar)
    in BreadcrumbBar (created by Connect(BreadcrumbBar))
    in Connect(BreadcrumbBar)
    in Provider
warning.js:33 Warning: Failed prop type: The prop `children` is marked as required in `EllipisWithTooltip`, but its value is `null`.
    in EllipisWithTooltip (created by BreadcrumbSwitcherPopover)
    in BreadcrumbSwitcherPopover (created by BreadcrumbSwitcher)
    in Position (created by Overlay)
    in Transition (created by Fade)
    in Fade (created by Overlay)
    in RootCloseWrapper (created by Overlay)
    in Portal (created by Overlay)
    in Overlay (created by Overlay)
    in Overlay (created by BreadcrumbSwitcher)
    in div (created by BreadcrumbSwitcher)
    in BreadcrumbSwitcher (created by BreadcrumbBar)
    in ol (created by Breadcrumb)
    in Breadcrumb (created by Breadcrumb)
    in Breadcrumb (created by Breadcrumb)
    in Breadcrumb (created by BreadcrumbBar)
    in div (created by BreadcrumbBar)
    in BreadcrumbBar (created by Connect(BreadcrumbBar))
    in Connect(BreadcrumbBar)
    in Provider

Expected results:
The switcher should display the same names as the index table.


Related issues

Related to Foreman - Bug #23527: enable breadcrumbs switcher in trends pagesClosed2018-05-07

Associated revisions

Revision 5bc5094b (diff)
Added by Amir Fefer 4 months ago

Fixes #23559 - fix breadcrumbs switcher in trends page

History

#1 Updated by Tomáš Strachota 4 months ago

  • Related to Bug #23527: enable breadcrumbs switcher in trends pages added

#2 Updated by The Foreman Bot 4 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/5551 added

#3 Updated by Tomáš Strachota 4 months ago

  • Legacy Backlogs Release (now unused) set to 353

#4 Updated by Amir Fefer 4 months ago

  • % Done changed from 0 to 100
  • Status changed from Ready For Testing to Closed

#5 Updated by Walden Raines 3 months ago

  • Target version deleted (219)

Also available in: Atom PDF