pmndrs / valtio

💊 Valtio makes proxy-state simple for React and Vanilla
http://valtio.pmnd.rs
MIT License
8.7k stars 244 forks source link

docs: Add Cache invalidation description in some-gotchas #721

Closed DevSDK closed 1 year ago

DevSDK commented 1 year ago

Related Issues or Discussions

The current implementation of caching in snapshot has a limitation with the array mutation #712 .

The unexpected behavior is hard to find or debug without proper description, especially since console.log and Redux devtools are not effective in this case. Before resolving this, this needs to be described as documentation.

Therefore, add the description in some-gotchas with example cases.

Summary

Add description about #712

Check List

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
valtio ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 19, 2023 0:11am
codesandbox-ci[bot] commented 1 year ago

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.

Latest deployment of this branch, based on commit 44bf5c783d7f9b5c6cd9babad07984757f17d834:

Sandbox Source
React Configuration
React Typescript Configuration
React Browserify Configuration
React Snowpack Configuration
React Parcel Configuration
DevSDK commented 1 year ago

@dai-shi Could you please check the merge button? I guess I'm not authorized to merge.

image
dai-shi commented 1 year ago

Thanks for your fix with format!