fix(addSourceWizard): copy updates (#26)

* copy updates for error handling
quipucords · Mar 25, 2019 · 2107682 · 2107682
1 parent 496df34
commit 2107682
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 8 deletions.
diff --git a/src/components/addSourceWizard/__tests__/__snapshots__/addSourceWizardStepThree.test.js.snap b/src/components/addSourceWizard/__tests__/__snapshots__/addSourceWizardStepThree.test.js.snap
@@ -53,7 +53,7 @@ exports[`AccountWizardStepResults Component should render a wizard results step
     <p
       className="blank-slate-pf-secondary-action"
     >
-      There are errors on the previous step
+      There are errors on a previous step. Use the Back button to review your settings and try again.
     </p>
   </div>
 </AddSourceWizardStepThree>
@@ -99,7 +99,7 @@ exports[`AccountWizardStepResults Component should render a wizard results step
     <p
       className="blank-slate-pf-secondary-action"
     >
-      There are errors on the previous step
+      There are errors on a previous step. Use the Back button to review your settings and try again.
     </p>
   </div>
 </AddSourceWizardStepThree>

diff --git a/src/components/addSourceWizard/addSourceWizardStepThree.js b/src/components/addSourceWizard/addSourceWizardStepThree.js
@@ -11,7 +11,9 @@ const AddSourceWizardStepThree = ({ add, error, fulfilled, pending, name }) => (
           <Icon type="pf" name="error-circle-o" />
         </div>
         <h3 className="blank-slate-pf-main-action">Error {add ? 'Creating' : 'Updating'} Source</h3>
-        <p className="blank-slate-pf-secondary-action">There are errors on the previous step</p>
+        <p className="blank-slate-pf-secondary-action">
+          There are errors on a previous step. Use the Back button to review your settings and try again.
+        </p>
       </div>
     )}
     {fulfilled && (

diff --git a/src/components/addSourceWizard/addSourceWizardStepTwo.js b/src/components/addSourceWizard/addSourceWizardStepTwo.js
@@ -137,7 +137,7 @@ class AddSourceWizardStepTwo extends React.Component {
       <FormField
         label="Name"
         error={(touched.name && errors.name) || stepTwoErrorMessages.name}
-        errorMessage={stepTwoErrorMessages.name || 'You must enter a source name '}
+        errorMessage={stepTwoErrorMessages.name || 'A source name is required'}
       >
         <Form.FormControl
           type="text"
@@ -201,7 +201,7 @@ class AddSourceWizardStepTwo extends React.Component {
             <FormField
               label="Search addresses"
               error={(touched.hostsMultiple && errors.hosts) || stepTwoErrorMessages.hosts}
-              errorMessage="You must enter a valid IP address or hostname"
+              errorMessage="A valid IP address or hostname is required"
             >
               <Form.FormControl
                 componentClass="textarea"
@@ -237,7 +237,7 @@ class AddSourceWizardStepTwo extends React.Component {
       case 'satellite':
         const hostPortError = (
           <React.Fragment>
-            {touched.hostsSingle && errors.hosts && 'You must enter a valid IP address or hostname '}
+            {touched.hostsSingle && errors.hosts && 'A valid IP address or hostname is required '}
             {errors.port && 'Port must be valid'}
           </React.Fragment>
         );
@@ -253,7 +253,7 @@ class AddSourceWizardStepTwo extends React.Component {
                 stepTwoErrorMessages.port
               }
               errorMessage={
-                (stepTwoErrorMessages.hosts && 'You must enter a valid IP address or hostname') ||
+                (stepTwoErrorMessages.hosts && 'A valid IP address or hostname is required') ||
                 (stepTwoErrorMessages.port && 'Port must be valid') ||
                 hostPortError
               }
@@ -296,7 +296,7 @@ class AddSourceWizardStepTwo extends React.Component {
       <FormField
         label="Credentials"
         error={(touched.credentials && errors.credentials) || stepTwoErrorMessages.credentials}
-        errorMessage={stepTwoErrorMessages.credentials || 'You must add a credential'}
+        errorMessage={stepTwoErrorMessages.credentials || 'A credential is required'}
       >
         <Form.InputGroup>
           <DropdownSelect