Skip to content

[MISC][HN] Fix async validation race condition in array fields with showIf #84

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.

Sign up for GitHub

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

Jump to bottom
Merged
weili-govtech merged 1 commit into from
Nov 19, 2025
Merged

[MISC][HN] Fix async validation race condition in array fields with showIf #84

weili-govtech merged 1 commit into from
Nov 19, 2025

Conversation

@nvhieu9
Copy link

@nvhieu9 nvhieu9 commented Nov 19, 2025

Changes

  • Use Yup.lazy() to create fresh schema instances per array item
  • Prevents schema mutation conflicts during parallel async validation
  • Add test case for async validation with multiple array items
  • [delete] branch

Changelog entry

  • Fix async validation race condition in array fields with showIf

@nvhieu9 nvhieu9 self-assigned this Nov 19, 2025
@nvhieu9 nvhieu9 requested a review from qroll November 19, 2025 09:29
@weili-govtech weili-govtech merged commit db65051 into main Nov 19, 2025
1 check failed
@weili-govtech weili-govtech deleted the CCUBE-2029 branch November 19, 2025 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@weili-govtech weili-govtech weili-govtech approved these changes

@qroll qroll Awaiting requested review from qroll

Assignees

@nvhieu9 nvhieu9

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nvhieu9 @weili-govtech