Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[fix-debounce]: should not send values repeatedly #47

Merged
merged 2 commits into from
Nov 8, 2019

Conversation

luoxiu
Copy link
Member

@luoxiu luoxiu commented Nov 8, 2019

fix #46

Sorry for my carelessness. 🤦‍♂️

@luoxiu
Copy link
Member Author

luoxiu commented Nov 8, 2019

debounce and throttle still have a lot of edging cases that need to be verified, I will focus on them this weekend after I have solved #37 .

@luoxiu luoxiu requested a review from ddddxxx November 8, 2019 03:10
@luoxiu luoxiu requested a review from ddddxxx November 8, 2019 08:34
@luoxiu luoxiu merged commit e4b7733 into master Nov 8, 2019
@luoxiu luoxiu deleted the fix-debounce-emitting-issue branch November 8, 2019 08:47
luoxiu added a commit that referenced this pull request Jun 1, 2021
[fix-debounce]: should not send values repeatedly
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Debounce keeps emitting values even though debounced publisher is not publishing
2 participants