Fix field not validating after re-mount (#3566, #3500)

[viljark]

Because we do

return (
    !structure.deepEqual(values, nextProps && nextProps.values) ||
    !structure.deepEqual(lastFieldValidatorKeys, fieldValidatorKeys)
  )

in the defaultShouldValidate, we have to remove the field keys from the lastFieldValidatorKeys when Field is destroyed, otherwise on re-mount the key still exists and validation is skipped.

Should fix:
#3500
#3566

[codecov-io]

Codecov Report

Merging #3909 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #3909   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files          70      70           
  Lines        1629    1631    +2     
======================================
+ Hits         1629    1631    +2

Impacted Files	Coverage Δ
src/createReduxForm.js	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 085b218...c33440d. Read the comment docs.

[asaarnak]

Hei @erikras @gustavohenke!
Could take a look at this PR please?
Quite important for us.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

[erikras]

Released in v7.4.0.