Missing error when validating via form options instead of via field #678

Activity

1. crutchcorn commented on Apr 17, 2024

   

   crutchcorn

   on Apr 17, 2024

   Contributor

   More actions

   Away from a computer right now. Are you checking against the field errors or the form errors. This is a bit confusing, but form validation like this doesn't set field errors, it sets form errors
2. bennettdams commented on Apr 17, 2024

   

   bennettdams

   on Apr 17, 2024

   Last edited by bennettdams

   ContributorAuthor

   More actions

   Are you checking against the field errors or the form errors.

   Yep, was using the FieldInfo helper from the examples.

   it sets form errors

   You're right, the validation message exists on field.form.state.errors, thanks!

   This is a bit confusing

   I have to agree that this is confusing, as one would always need to look for errors in multiple places.
   Also, it seems that the form has no touchedErrors like a Field has?

   Helper component for reference:

   function FieldInfo({ field }: { field: FieldApi<any, any, any, any> }) {
     // This is empty when using the "form" validator (but not for the "field" validator)
     console.log("field touchedErrors", field.state.meta.touchedErrors);
     // ...but not empty for the form's `errors`
     console.log("form errors", field.form.state.errors);
     return (
       <>
         {field.state.meta.touchedErrors.length === 0 ? (
           "No errors"
         ) : field.state.meta.touchedErrors ? (
           <em>{field.state.meta.touchedErrors}</em>
         ) : null}
       </>
     );
   }
3. crutchcorn commented on Apr 18, 2024

   

   crutchcorn

   on Apr 18, 2024

   Contributor

   More actions

   FWIW we're actually planning on removing touchedErrors 😅

   Also, we're gonna make the story of form->field validation easier here:

   #656

   Closing as both are a WIP, but thanks for the feedback and detailed report 😇

   👍React with 👍1Reacted by Bennett Dams
4. crutchcorn

   closed this as completedon Apr 18, 2024