search

openshift-assisted / assisted-installer-ui

The Assisted Installer User Interface
https://console.redhat.com/openshift/create/datacenter
Apache License 2.0
9 stars 37 forks source link

MGMT-15667: Clear base domain last error message when changing the textbox content #2318

Closed ammont82 closed 1 year ago

ammont82 commented 1 year ago

Related to https://issues.redhat.com/browse/MGMT-15667

In cluster details form:

  1. User may write invalid base_domain press next and got warning returned.
  2. Once the user re-enter the text box and change the content (before pressing next) the warning is not relevant anymore and needs to be removed.
openshift-ci-robot commented 1 year ago

@ammont82: This pull request references MGMT-15667 which is a valid jira issue.

In response to [this](https://github.com/openshift-assisted/assisted-installer-ui/pull/2318): >Related to https://issues.redhat.com/browse/MGMT-15667 > >In cluster details form: >1. User may write invalid base_domain press next and got warning returned. >2. Once the user re-enter the text box and change the content (before pressing next) the warning is not relevant anymore and needs to be removed. Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
openshift-ci[bot] commented 1 year ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ammont82, jgyselov, jkilzi

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/openshift-assisted/assisted-installer-ui/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
openshift-ci[bot] commented 1 year ago

New changes are detected. LGTM label has been removed.