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

Define behavior of changes to initialState value #1754

Closed
wants to merge 1 commit into from

Conversation

rubencodes
Copy link

This PR adds information about what happens if the initialState argument to useState changes value on subsequent re-renders (which is: nothing, it's ignored).

@facebook-github-bot
Copy link
Collaborator

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need the corporate CLA signed.

If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks!

@reactjs-bot
Copy link

Deploy preview for reactjs ready!

Built with commit 97e5ce4

https://deploy-preview-1754--reactjs.netlify.com

@facebook-github-bot
Copy link
Collaborator

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@BartoszKlonowski
Copy link
Collaborator

Hello @rubencodes,
this is a valid note, thank you for the contribution!
Looking at the latest docs describing the useState hook I see that the fact of ignoring the further changes to initial state is already mentioned in Avoiding recreating the initial state section.
Let me close this PR as outdated, however please feel encouraged to create new PRs for open issues.

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.

4 participants