docs - Added a note in how-tos/reset on how to use structuredClone #947
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
More templates
commit: |
docs/how-tos/how-to-reset-state.mdx
Outdated
| @@ -34,3 +34,36 @@ const reset = () => { | |||
| state.obj = _.cloneDeep(initialObj) | |||
There was a problem hiding this comment.
I mean this is wrong too. L31-35
Can you please fix the doc in this way?
- Fix cloneDeep usage with v2.
- Add a note about structuredClone
Speaking of v2, we now have deepClone from valtio/utils, we don't need to use cloneDeep.
So, we could restructure the docs to use deepClone first and structuredClone second.
(I thought about explaining structuredClone first, but deepClone is capable of avoid copying ref()'d object.
There was a problem hiding this comment.
Sure thing. I'll push up the changes soon.
There was a problem hiding this comment.
Changes have been pushed. Let me know of any feedback.
Related Bug Reports or Discussions
N/A
Fixes #
N/A
Summary
Added example on how to use structuredClone in v2 and the reasoning behind why it's different than v1.
Check List
pnpm run prettierfor formatting code and docs