Project

General

Profile

Actions

Bug #23248

closed

Breadcrumbs switcher uses wrong ids on compute attributes detail page

Added by Tomáš Strachota about 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
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 2 (1 open1 closed)

Related to Foreman - Feature #22855: Add redux container for breadcrumb switcherClosedAmir Fefer03/12/2018Actions
Related to Foreman - Bug #23525: add a breadcrumbs switcher in compute attribute pageNew05/07/2018Actions
Actions #1

Updated by Tomáš Strachota about 6 years ago

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

Updated by Amir Fefer about 6 years ago

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

Updated by The Foreman Bot about 6 years ago

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

Updated by Ohad Levy about 6 years ago

  • translation missing: en.field_release set to 330

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

Actions #5

Updated by Amir Fefer about 6 years ago

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

Updated by The Foreman Bot almost 6 years ago

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

Updated by Walden Raines almost 6 years ago

  • Target version deleted (219)
Actions

Also available in: Atom PDF