Feature #8831

Turn discovered hosts show form to two-pane

Added by Ori Rabin almost 3 years ago. Updated over 1 year ago.

Status:Rejected
Priority:Normal
Assigned To:Ori Rabin
Category:Discovery plugin
Target version:Plugin 4.0.0
Difficulty: Pull request:https://github.com/theforeman/foreman_discovery/pull/113
Bugzilla link:
Story points-
Velocity based estimate-

Description

The show form for discovered hosts only shows the facts and actions that are available in the index.
The show can change to a two-pane display from the index page with or without the actions of delete, refresh facts etc.


Related issues

Blocked by Discovery - Bug #9017: [2.1] Remove backward compatibility version workarounds Closed 01/19/2015

Associated revisions

Revision 7a645470
Added by Ori Rabin almost 3 years ago

Fixes #8831 - Turn discovered hosts show form to two-pane

Revision 6bbd8f69
Added by Lukas Zapletal almost 3 years ago

Refs #8831 - Reverts two-pane forms

We are pulling two-pane patch off the 2.0 for now. It's too complicated for
now. Let's do proper implementation for Foreman 1.8 later on.

This reverts commit 7a6454704cf87f712d9f32fae584245c41f24fa1.

Conflicts:
app/views/discovered_hosts/_discovered_host.html.erb
app/views/discovered_hosts/_discovered_hosts_list.html.erb

History

#1 Updated by The Foreman Bot almost 3 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_discovery/pull/113 added

#2 Updated by Ori Rabin almost 3 years ago

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

#3 Updated by Lukas Zapletal almost 3 years ago

  • Status changed from Closed to New

The patch was reverted for 2.0.0.

https://github.com/theforeman/foreman_discovery/pull/126

Let's review this after 2.0.0 release.

#4 Updated by Ori Rabin almost 3 years ago

  • Blocked by Bug #9017: [2.1] Remove backward compatibility version workarounds added

#5 Updated by Ori Rabin almost 3 years ago

#6 Updated by Ori Rabin almost 3 years ago

  • Status changed from New to Assigned

#7 Updated by The Foreman Bot almost 3 years ago

  • Status changed from Assigned to Ready For Testing

#8 Updated by Lukas Zapletal over 2 years ago

  • Target version set to Plugin 3.0.0

#9 Updated by Lukas Zapletal over 2 years ago

  • Related to deleted (Tracker #9018: Foreman Discovery 2.1 features)

#10 Updated by Lukas Zapletal over 2 years ago

  • Target version changed from Plugin 3.0.0 to Plugin 4.0.0

#11 Updated by Ori Rabin over 1 year ago

  • Status changed from Ready For Testing to Rejected

This is no longer relevant after UI changes that extended the show form.
Fitting it into two-pane now will not look so great.

Also available in: Atom PDF