Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UNREGISTER_FIELD sync error issue #3433

Closed
iliesalex89 opened this issue Sep 20, 2017 · 6 comments
Closed

UNREGISTER_FIELD sync error issue #3433

iliesalex89 opened this issue Sep 20, 2017 · 6 comments
Labels
needs sandbox missing codesandbox link

Comments

@iliesalex89
Copy link

Starting from version 7.0.2, I discovered that issuess fixed in the past regarding removing sync errors when a field is unregistered are back in current behavior. I think this thing appeared after "Fixed tricky bug with UNREGISTER action not removing deep sync errors. #3219 #3189".

I have now a form with destroyOnMount false and when I'm trying to getFormSyncErrors for fields that are not registered, this is not possible. I think my issue is closely related to issues raised in the recent past like:
"Do not clear errors unless field is being removed from the form #3064"
"[6.7.0] Unregister field shouldn't clean up its sync error if there are multiple fields under this name #2979"

@bharadwajsampath
Copy link

Even I have the same issue.
I am unregistering a field but the sync error for that field happens to be present in the reduxForm syncerrors state.

@gustavohenke
Copy link
Collaborator

Do you guys still happen to have this issue? If so, can you provide a sandbox that replicates it with the latest version?

@gustavohenke gustavohenke added the needs sandbox missing codesandbox link label Jan 31, 2018
@gustavohenke
Copy link
Collaborator

Closing due to inactivity.

@mitk0936
Copy link

The bug is still reproducible in 7.3.0

@aldoshin
Copy link

I tried explicitly dispatching unregisterField(SCHEMA_FORM_NAME, id, true) when the component containing the Field componentWillUnmount and worked seems some issue the way the third parameter destroyOnUnmount is handled

@lock
Copy link

lock bot commented Apr 2, 2020

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.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs sandbox missing codesandbox link
Projects
None yet
Development

No branches or pull requests

5 participants