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

[Bug]Changes to items in FieldArray cause whole list of items to update #3791

Closed
jarekb84 opened this issue Jan 29, 2018 · 2 comments · Fixed by #3803
Closed

[Bug]Changes to items in FieldArray cause whole list of items to update #3791

jarekb84 opened this issue Jan 29, 2018 · 2 comments · Fixed by #3803

Comments

@jarekb84
Copy link

jarekb84 commented Jan 29, 2018

Are you submitting a bug report or a feature request?

Bug

What is the current behavior?

Changing a single item in a FieldArray causes whole list to be resorted. Inspecting actions and state diff using Redux dev tools shows an action dispatched where payload updates the correct item (via index key), but the difference in state includes changes to all items in the list.

What is the expected behavior?

Changing a single item only updates that item's state in the redux store.

Sandbox Link

Attempted to create sandbox with scenario, but there are issues with the current sandbox setup with 7.2.1
https://codesandbox.io/s/811pjkq2y9

What's your environment?

"redux-form": "7.2.1"
"redux": "^3.5.2"
"immutable": "^3.8.1"
"react-redux": "^5.0.1",
Chrome 63.0.3239.132
Node 8.9.1

Other information

Note this does not happen in version 7.0.4/7.1.2 and when number of items provided to FieldArray is less than 32.

@erikras
Copy link
Member

erikras commented Feb 1, 2018

Published fix in v7.2.2.

@lock
Copy link

lock bot commented Feb 1, 2019

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 Feb 1, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants