remove superfluous useCallback

[kentcdodds]

When people see useCallback used like this without explanation for why, they'll just assume they should put all callbacks within a useCallback which is not a good thing.

[danielkcz]

Maybe I am still misunderstanding something conceptual, but I've used the Form there on purpose to signify there might be some more complex logic behind it. For example, if you would have useEffect inside the Form component that would for some reason need to depend on onSubmit, it would cause unnecessary re-runs of that effect without useCallback in parent. Isn't that correct?

[kentcdodds]

My point is that there's nuance for when it's appropriate to use useCallback and given that the nuance is not present here, including it is misleading.

[danielkcz]

Ok fair enough, it should be probably explained if used.