Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug causing errors to be cleared when using autoProgress #578

Conversation

thomascullen
Copy link
Member

When setting a value for an input, we re run validation for any fields that have errors. This was reading from the errors state directly which can be out of date if called in the middle of a render. By using the setErrors callback syntax we will get the most up to date values.

I've also removed a line which skipped validating the current field in this setValue call. I'm not sure why this existed in the first place.

…values

When setting a value for an input, we re run validation for any fields
that have errors. This was reading from the errors state directly which
can be out of date if called in the middle of a render. By using the
setErrors callback syntax we will get the most up to date values.

I've also removed a line which skipped validating the current field in
this setValue call. I'm not sure why this existed in the first place.
@thomascullen thomascullen requested a review from a team as a code owner February 11, 2025 09:46
Copy link

changeset-bot bot commented Feb 11, 2025

🦋 Changeset detected

Latest commit: 523707f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
shared Patch
@evervault/ui-components Patch
@evervault/ui-components-e2e-tests Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@thomascullen thomascullen merged commit c967317 into master Feb 11, 2025
5 checks passed
@thomascullen thomascullen deleted the thomas/etr-3035-validation-doesnt-occur-when-autoprogress-is-set-to-true branch February 11, 2025 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants