Open rapzo opened 3 years ago
Ran exactly into the same issue... 😉
@kevinlandsberg @rapzo same issue! couldn't figure it out for a while, did you guys find any good solutions? thanks!
if you change values, by updating initialValue, you will need to set another property to formik enableReinitialize: true
. This way it will work as expected
There is a bug that occurs when the initial values is constructed directly inside the parameter object of the use formik function. It recreates the object every time it validates, and every time it creates an object it validates again. The solution was to create this initial values object inside a useEffect.
Example of a case where the problem occurs:
if you change values, by updating initialValue, you will need to set another property to formik
enableReinitialize: true
. This way it will work as expected
This worked! Thanks @ktalalykin .
Thank you @ktalalykin ! Fixed the issue
In my case @LeonardoDaLuz 's was the right approach, memoizing the initialData made more sense than enabling a reinitialization setting. Thanks
Bug report
Current Behavior
When using Formik with
validateOnMount: true
and mutatedinitialValues
controlled outside the form context, on the 1st change, Formik will go into an infinite validation loop, crashing the page eventually. This behavior is easily caught when using a toggle in the form that affects its validation and/or structure.Expected behavior
It is expected that, whenever
initialValues
change, a proper validation is done on the newly provided values.Reproducible example
Formik validateOnMount with a toggle that freezes the app: https://codesandbox.io/s/github/rapzo/formik-validateOnMount/tree/csb-dengj
A possible workaround, although it kind of limits the depth of the changeable value: https://codesandbox.io/s/github/rapzo/formik-validateOnMount/tree/main
Suggested solution(s)
https://github.com/formium/formik/compare/master...rapzo:patch-1
Additional context
I don't know if this was an expected behavior so I'm creating the issue before a PR. Will be happy to add test cases to the PR if this proposal makes sense.
Your environment