Bug 1324967
| Summary: | [RFE] Mark required fields conditionally based on other fields | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat Satellite | Reporter: | Og Maciel <omaciel> | ||||
| Component: | WebUI | Assignee: | satellite6-bugs <satellite6-bugs> | ||||
| WebUI sub component: | Foreman | QA Contact: | Katello QA List <katello-qa-list> | ||||
| Status: | CLOSED WONTFIX | Docs Contact: | |||||
| Severity: | medium | ||||||
| Priority: | low | CC: | bbuckingham, bkearney, inecas, mhulan, tbrisker | ||||
| Version: | 6.2.0 | Keywords: | FutureFeature, PrioBumpQA, Triaged | ||||
| Target Milestone: | Unspecified | ||||||
| Target Release: | Unused | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2018-07-17 19:19:32 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Attachments: |
|
||||||
The puppet environment is required field only when a puppet master is selected on the host. http://projects.theforeman.org/issues/7450. Given the current complexity of the host form UI, I don't think the gain for making sure we flag the environment as required only when puppet master is selected is worth the effort (even not sure it would get accepted upstream). Given the context, I suggest keeping this for sat-future and addressing it as part of bigger UI facelift with some better javascript plumbing around. Since it's not just a missed flag on obviously required field and there are reasons why it's not flagged by default, I don't agree on the blocker flag here. I think this won't be implemented until we have a way of specifying required fields validations based on other fields selections. That sounds more like a new way of rendering forms. Also it should not be addressed just for this field but rather every field, e.g. subnet has some fields required for ipv4 only. Converting to a more generic RFE Created redmine issue http://projects.theforeman.org/issues/21678 from this bug Thank you for your interest in Satellite 6. We have evaluated this request, and we do not expect this to be implemented in product in the forseeable future. We are therefore closing this out as WONTFIX. If you have any concerns about this, please feel free to contact Rich Jerrido or Bryan Kearney. Thank you. |
Created attachment 1144819 [details] Mark puppet environment as a required field This screenshot shows that "Puppet Environment" is a required field for provisioning new hosts. However, the form does not flag this field as being required. Other required fields have an "*" next to them to denote them as being required.