Project

General

Profile

Bug #23248

Breadcrumbs switcher uses wrong ids on compute attributes detail page

Added by Tomáš Strachota 6 months ago. Updated 3 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 5 months ago

Fixes #23248 - disable switcher in compute attributes

History

#1 Updated by Tomáš Strachota 6 months ago

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

#2 Updated by Amir Fefer 5 months ago

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

#3 Updated by The Foreman Bot 5 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 5 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 5 months ago

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

#6 Updated by The Foreman Bot 5 months ago

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

#7 Updated by Walden Raines 4 months ago

  • Target version deleted (219)

Also available in: Atom PDF