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

docs: Update third-party-libraries.md #1624

Closed
wants to merge 4 commits into from
Closed

docs: Update third-party-libraries.md #1624

wants to merge 4 commits into from

Conversation

AwesomeDevin
Copy link
Contributor

Update third-party-libraries:

  • zustand-vue - State management for vue (Vue3 / Vue2) based on zustand.
  • zustand-pub - Cross-Application/Cross-Framework State Management And Sharing based on zustand and zustand-vue for React/Vue.

zustand-vue - State management for vue (Vue3 / Vue2) based on zustand.
zustand-pub - Cross-Application/Cross-Framework State Management And Sharing based on zustand and zustand-vue for React/Vue.
@codesandbox-ci
Copy link

codesandbox-ci bot commented Feb 13, 2023

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 e6fd447:

Sandbox Source
React Configuration
React Typescript Configuration
React Browserify Configuration
React Snowpack Configuration
React Parcel Configuration
@pavlobu/zustand demo Configuration

Copy link
Member

@dai-shi dai-shi left a comment

Choose a reason for hiding this comment

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

Thanks. Can you put them in the alphabetical order?

@AwesomeDevin
Copy link
Contributor Author

alphabetical order

OK

edit in alphabetical order.
@AwesomeDevin
Copy link
Contributor Author

AwesomeDevin commented Feb 13, 2023

Thanks. Can you put them in the alphabetical order?

It’s done~

Comment on lines 47 to 48
- [zustand-rx](https://github.com/patdx/zustand-rx) — A Zustand middleware enabling you to subscribe to a store as an RxJS Observable.
- [zustand-querystring](https://github.com/nitedani/zustand-querystring) — A Zustand middleware that syncs the store with the querystring.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- [zustand-rx](https://github.com/patdx/zustand-rx) — A Zustand middleware enabling you to subscribe to a store as an RxJS Observable.
- [zustand-querystring](https://github.com/nitedani/zustand-querystring) — A Zustand middleware that syncs the store with the querystring.

@dai-shi
Copy link
Member

dai-shi commented Feb 13, 2023

Can you adjust them too?

@AwesomeDevin
Copy link
Contributor Author

Can you adjust them too?

Please review~

Copy link
Member

@dai-shi dai-shi left a comment

Choose a reason for hiding this comment

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

Thanks for your help!

Co-authored-by: Daishi Kato <dai-shi@users.noreply.github.com>
@AwesomeDevin
Copy link
Contributor Author

Thank you. I submitted your code

@dai-shi
Copy link
Member

dai-shi commented Feb 13, 2023

Hmmm, not sure what CI is complaining. Can you run yarn run prettier?

@NervJS NervJS closed this by deleting the head repository Feb 14, 2023
@AwesomeDevin
Copy link
Contributor Author

AwesomeDevin commented Feb 14, 2023

Hmmm, not sure what CI is complaining. Can you run yarn run prettier?

👌, I submitted a new pr

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.

3 participants