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.

Sign up for GitHub

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

Jump to bottom

debounce should support multi instances #145

Closed

Shemesh opened this issue Jul 7, 2022 · 3 comments

Assignees

Labels

bug

Milestone

Shemesh commented Jul 7, 2022

hi,
the debounce decorator does not support multi instances.
plz read this article - https://engineering.datorama.com/be-aware-of-the-debounce-decorator-6fb24a6d8d5
and here is the solution -https://github.com/NetanelBasal/helpful-decorators/blob/master/src/debounce.ts

Owner

vlio20 commented Jul 10, 2022

@Shemesh would you mind submitting a PR?

Owner

vlio20 commented Jul 15, 2022

The issue reproduced on stackblitz:
https://angular-3bpkw9.stackblitz.io

vlio20 self-assigned this

vlio20 added the bug label

vlio20 added this to the 2.0.5 milestone

Owner

vlio20 commented Jul 16, 2022

Will be available in 2.0.5
fixed in #146

vlio20 closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment