Project

General

Profile

Bug #23248

Breadcrumbs switcher uses wrong ids on compute attributes detail page

Added by Tomáš Strachota about 1 year ago. Updated 9 months ago.

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

Description

Breadcrumbs on the compute attributes detail page are tricky. Switcher lists compute profiles, but in fact the page url requires compute attribute id. As a result, the switcher links use compute profile ids in links to compute attributes. The links then lead to a seemingly random compute attributes (often to non-existent ones).

The eract switcher component always expects ids to be in 'id' attribute. One possible solution could be adding 'idField' attribute (we already have 'nameFiled') and enhance the compute profiles api with 'compute_attribute_id'. Alternatively we could use compute attributes api and add 'compute_profile_name'.


Related issues

Related to Foreman - Feature #22855: Add redux container for breadcrumb switcherClosed2018-03-12
Related to Foreman - Bug #23525: add a breadcrumbs switcher in compute attribute pageNew2018-05-07

Associated revisions

Revision 3d5cf7c9 (diff)
Added by Amir Fefer 12 months ago

Fixes #23248 - disable switcher in compute attributes

History

#1 Updated by Tomáš Strachota about 1 year ago

  • Related to Feature #22855: Add redux container for breadcrumb switcher added

#2 Updated by Amir Fefer 12 months ago

  • Related to Bug #23525: add a breadcrumbs switcher in compute attribute page added

#3 Updated by The Foreman Bot 12 months ago

  • Assignee set to Amir Fefer
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/5537 added

#4 Updated by Ohad Levy 12 months ago

  • Legacy Backlogs Release (now unused) set to 330

setting to 1.18 as I believe we should cherry-pick this.

#5 Updated by Amir Fefer 12 months ago

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

#6 Updated by The Foreman Bot 11 months ago

  • Pull request https://github.com/theforeman/foreman/pull/5593 added

#7 Updated by Walden Raines 10 months ago

  • Target version deleted (219)

Also available in: Atom PDF