Skip to content

Update Zustand code example#439

Open
dbritto-dev wants to merge 1 commit into
statelyai:mainfrom
dbritto-dev:main
Open

Update Zustand code example#439
dbritto-dev wants to merge 1 commit into
statelyai:mainfrom
dbritto-dev:main

Conversation

@dbritto-dev

Copy link
Copy Markdown

No description provided.

@vercel

vercel Bot commented Sep 22, 2024

Copy link
Copy Markdown

@dbritto-dev is attempting to deploy a commit to the Stately Team on Vercel.

A member of the Team first needs to authorize it.

@davidkpiano

Copy link
Copy Markdown
Member

Thanks for the PR! Can you add this as a separate section, since it isn't the primary/idiomatic usage of Zustand?

@dbritto-dev

Copy link
Copy Markdown
Author

@davidkpiano below? or somewhere else?

@dbritto-dev

Copy link
Copy Markdown
Author

@davidkpiano btw, we want to promote that. Currently, we are working on revamping docs. I added docs for APIs and Hooks for now.

@dbritto-dev

Copy link
Copy Markdown
Author

@davidkpiano ?

@davidkpiano

davidkpiano commented Sep 24, 2024

Copy link
Copy Markdown
Member

@davidkpiano below? or somewhere else?

Below is fine.

EDIT: Also, I would like this section to reflect what is seen here, as this is where I'm referencing the examples in the documentation: https://zustand.docs.pmnd.rs/getting-started/comparison

@dbritto-dev

Copy link
Copy Markdown
Author

@davidkpiano sure, let me update it

@davidkpiano

Copy link
Copy Markdown
Member

@davidkpiano sure, let me update it

Hey @dbritto-dev - I'd like to update the comparison but it seems that the Zustand docs are still pointing to the old way (no inferred types).

@dbritto-dev

Copy link
Copy Markdown
Author

@davidkpiano sure, let me update it

Hey @dbritto-dev - I'd like to update the comparison but it seems that the Zustand docs are still pointing to the old way (no inferred types).

Yeah, we are doing some housekeeping right now removing outdated links but I'll work on that after cleaning/updating some docs.

BTW, we talk about combine in the docs api section

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