Skip to content

Add key warning to nested collections#3758

Merged
sophiebits merged 1 commit intofacebook:masterfrom
sophiebits:keys-keys-keys
Apr 27, 2015
Merged

Add key warning to nested collections#3758
sophiebits merged 1 commit intofacebook:masterfrom
sophiebits:keys-keys-keys

Conversation

@sophiebits
Copy link
Copy Markdown
Collaborator

Also when reusing elements in multiple contexts -- before we were mutating each element to indicate its validity; now we mutate the array containing it (which we create, in the case of rest-arg children).

Fixes #2496. Fixes #3348.

Also when reusing elements in multiple contexts -- before we were mutating each element to indicate its validity; now we mutate the array containing it (which we create, in the case of rest-arg children).

Fixes facebook#2496. Fixes facebook#3348.
sophiebits added a commit that referenced this pull request Apr 27, 2015
Add key warning to nested collections
@sophiebits sophiebits merged commit d359319 into facebook:master Apr 27, 2015
zpao added a commit to zpao/react that referenced this pull request May 7, 2015
zpao added a commit to zpao/react that referenced this pull request May 8, 2015
zpao added a commit that referenced this pull request May 8, 2015
Followup to #3758 so we actually test things
framp pushed a commit to framp/react that referenced this pull request May 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Nested arrays without key don't warn (anymore?) Make sure we do the child key warning for all cases

4 participants