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

fix: cache snapshot & add option deps #30

Merged
merged 2 commits into from
Sep 23, 2024

Conversation

yuyi919
Copy link
Contributor

@yuyi919 yuyi919 commented Sep 23, 2024

fix #22

Adding the deps option is to allow users to manually control whether to detect changes in the keys (references), saving the trouble of using useMemo. This might be a better default behavior, but it could introduce a breaking change.

@ai ai merged commit ff27de9 into nanostores:main Sep 23, 2024
2 of 3 checks passed
@ai
Copy link
Member

ai commented Sep 23, 2024

Thanks. Release in 0.8.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error for useStore: Warning: The result of getSnapshot should be cached to avoid an infinite loop
2 participants