Fix validation on stale values

[Jestermaxrko]

Fixes issue - #2083

The cause of the issue is in this line

formik/packages/formik/src/Formik.tsx

Line 185 in 0f960aa

const state = stateRef.current;

beause of assigning value from stateRef.current to the state variable this variable contains "old" value until next re-render

formik/packages/formik/src/Formik.tsx

Line 190 in 0f960aa

stateRef.current = formikReducer(prev, action);

And then when setFieldValue and setFieldTouched are called consecutively stateRef.current references to updated value imediatelly after setFieldValue call, but state variable is still references to the old value and setFieldTouched runs validation on that old value

Using stateRef.current directly gurantees that actual values are taken

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
formik-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 11, 2024 10:40pm

[changeset-bot]

🦋 Changeset detected

Latest commit: e004b64

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

This PR includes changesets to release 2 packages

Name	Type
formik	Patch
formik-native	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

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

[ypk4]

Helpful change. Are we pending one more review to get this merged?

[pvcresin]

Thank you very much. I hope it will be released.

[quantizor]

Would love a test please

[quantizor]

I think I see some of the source of confusion, in Formik.tsx stateRef.current is saved to state but the issue is that ref is wholly-replaced inside of dispatch. We should probably remove that variable altogether because it is a quiet source of bugs.

formik/packages/formik/src/Formik.tsx

Line 185 in c6ceb65

const state = stateRef.current;

[gone-skiing]

Any chance this can get merged and released? Seems like a helpful change to keep on the shelf.