[go: up one dir, main page]

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

Add redux-debounce analog #37

Open
ai opened this issue Apr 30, 2019 · 4 comments
Open

Add redux-debounce analog #37

ai opened this issue Apr 30, 2019 · 4 comments

Comments

@ai
Copy link
Member
ai commented Apr 30, 2019

https://github.com/ryanseddon/redux-debounced

@igorkamyshev
Copy link
Contributor
igorkamyshev commented May 8, 2019

Hello. I try to implement it, but have one issue.

If the event was recently, I have to stop all subsequent ones, but storeon "middlewares" can't destroy actions.

Can you give me advice, how I may implement it?

@ai
Copy link
Member Author
ai commented May 8, 2019
  1. We can suggest debounce for event listeners: store.on('event', debounce(…)). In this case, users can use any debounce library.
  2. We can improve the core API for this task. For instance, by returning false.

@igorkamyshev
Copy link
Contributor
  1. In this case, we don't need specific package for debounce, I guess.
  2. If we use solution from "1" to debounce actions, maybe anyway implement this core API? I think, it may be useful for many reasons.

@ai
Copy link
Member Author
ai commented May 8, 2019

I will think about new API to make debounce, undo and Logux support after May 11

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

No branches or pull requests

2 participants