-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Warn in toJS
docs about deep cloning partial observables
#3239
Conversation
Warn in `toJS` docs about deep cloning partial observables, to avoid issues with some properties being deep clones while others are not. Relates to mobxjs#2912
🦋 Changeset detectedLatest commit: c7fd2a5 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@bernardobelchior Thank you for the clarification. Would you like to add this information to Mobx docs too? https://mobx.js.org/api.html#tojs |
@kubk I wouldn't mind. Although we would have to merge the docs and this comment, since they overlap a bit. |
@bernardobelchior OK, feel free to create a separate PR to Mobx docs. Thank you. @urugator Is there a way to restart pipelines? |
@kubk Yes by closing and reopening the PR :D |
Warn in
toJS
docs about deep cloning partial observables, to avoid issues with some properties being deep clones while others are not.I have faced this issue while upgrading to MobX 6 and I believe it will save other people some headaches if it is clear that
toJS
has this limitation.Relates to #2912