-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Add common immutable update recipes and libraries #525
Conversation
Deploy preview for reactjs ready! Built with commit 8fad11d |
content/docs/immutable-updates.md
Outdated
|
||
### Updating Object Properties | ||
|
||
To update an object property, you can make use of the [spread syntax](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/assign): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should reorder these sections. Show Object.assign
first since it's in the standard, then mention a syntax proposal (but also mention it's enabled by default in Create React App, and already ships in Node and Chrome).
@nateeo I can't find your repo in the list of forks so I can't make a PR to you fork. I also can't push directly to this PR branch. Any ideas? |
my bad, should have sent you a link earlier!
…On 30/01/2018 02:38, "Ernesto García" ***@***.***> wrote:
@nateeo <https://github.com/nateeo> I can't find your repo in the list of
forks so I can't make a PR to you fork. I also can't push directly to this
PR branch. Any ideas?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#525 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAljrfh0Qa_Oww9ndAGUqlbXDkpxAx4zks5tPcnlgaJpZM4ReEle>
.
|
@gnapse did you get the link? |
TBH, I do not fully recall about this right now. |
work in progress!
resolves #498 (see for discussion)