Project

General

Profile

Bug #7560

add :required => true on host fields that have conditional validation if host.managed?

Added by Joseph Magen over 8 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Joseph Magen
Category:
Web Interface
Target version:
1.7.0
Difficulty:
Triaged:
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/1783
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

The New Host defaults to a Managed Host, so the UI should display the following fields as required, since there is no option in the UI to create an unmanaged host.

architecture_id
operatingsystem_id
domain_id
mac - when a computer resource is selected, this field is hidden anyway

Related issues

Related to Foreman - Bug #7450: We mark not-required fields as requiredClosed2014-09-16
Related to Foreman - Bug #7750: Foreman - Failed to "Submit" while building a new VMClosed2014-09-30

Associated revisions

Revision 571614ae (diff)
Added by Joseph Magen over 8 years ago

fixes #7560 - add :required => true on host fields that have conditional validation if host.managed?

History

#1 Updated by The Foreman Bot over 8 years ago

  • Status changed from Assigned to Ready For Testing
  • Target version set to 1.7.3
  • Pull request https://github.com/theforeman/foreman/pull/1783 added
  • Pull request deleted ()

#2 Updated by Dominic Cleal over 8 years ago

  • Related to Bug #7450: We mark not-required fields as required added

#3 Updated by Daniel Lobato Garcia over 8 years ago

  • Legacy Backlogs Release (now unused) set to 21

#4 Updated by Daniel Lobato Garcia over 8 years ago

In the end some more fields were added, 

Operating System
        Provisioning Method
        Media
        Partition table
        Root password

#5 Updated by Joseph Magen over 8 years ago

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

#6 Updated by Dominic Cleal over 8 years ago

  • Related to Bug #7750: Foreman - Failed to "Submit" while building a new VM added

Also available in: Atom PDF