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

chore(explorer): setup zustand with context #3077

Merged
merged 4 commits into from
Aug 28, 2024

Conversation

karooolis
Copy link
Contributor

@karooolis karooolis commented Aug 28, 2024

Previous implementation was possibly prone to SSR-related issues. Updated zustand setup according to the official guide: https://zustand.docs.pmnd.rs/guides/nextjs

Copy link

changeset-bot bot commented Aug 28, 2024

⚠️ No Changeset found

Latest commit: cb7624d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Member

@holic holic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oof so much boilerplate! but glad there's a solution, I am pretty sure I've come across this too

"app store" makes me think of iOS/MacOS but I don't have a good alternative to suggest 🤷

@karooolis
Copy link
Contributor Author

oof so much boilerplate! but glad there's a solution, I am pretty sure I've come across this too

"app store" makes me think of iOS/MacOS but I don't have a good alternative to suggest 🤷

Same thoughts about "app store" :/ considered using "app data store" but too long. Might come up with a better name eventually.

@karooolis karooolis merged commit b3361b9 into main Aug 28, 2024
14 checks passed
@karooolis karooolis deleted the kumpis/zustand-context-store branch August 28, 2024 11:54
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.

2 participants