fix: remove function fired in useEffect from dependency array

[dsowinski2]

No description provided.

[rafalcymerys]

@dsowinski2 could you elaborate a bit on this one? It's not clear to me what's the purpose of this change

[dsowinski2]

@dsowinski2 could you elaborate a bit on this one? It's not clear to me what's the purpose of this change

Sure! There was function 'reserForm' included in dependency array of useEffect, which in fact called resetForm function inside. So that created infinite loop, producing errors in console. It's not needed here as we should only reset form if it's open/closed

[rafalcymerys]

@dsowinski2 could you elaborate a bit on this one? It's not clear to me what's the purpose of this change

Sure! There was function 'reserForm' included in dependency array of useEffect, which in fact called resetForm function inside. So that created infinite loop, producing errors in console. It's not needed here as we should only reset form if it's open/closed

Do you have any steps to reproduce? I'm trying to understand where this is coming from and why we haven't seen it while testing.