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

msg.reset doesn't work as supposed to #10

Open
ghost opened this issue Mar 17, 2019 · 4 comments
Open

msg.reset doesn't work as supposed to #10

ghost opened this issue Mar 17, 2019 · 4 comments
Assignees
Labels

Comments

@ghost
Copy link

ghost commented Mar 17, 2019

msg.reset doesn't reset the counter to it's initial count and instead keeps on counting, nor does it reset to 0 when the msg.reset value is the number 0.
Not sure if that's a bug but it can be verified with the examples in the repository.

node-red-counter 0.1.5
node-red 020.2
node.js 8.15.0

@dezral
Copy link

dezral commented Jun 6, 2019

i have the same problem.. i have to delete the counter, and make a new..

@dkern dkern self-assigned this Jun 12, 2019
@dkern dkern added the bug label Jun 12, 2019
@dkern
Copy link
Owner

dkern commented Jun 12, 2019

Well, it seems that using msg.reset = 0 is buggy and don't reset anything. If the default value is 0 and you want to reset, just use msg.reset = true, this will reset the value to zero then. Anything above zero can be set with msg.reset = 123.

I will fix that bug when i have time. ;)

@ondrackajan
Copy link

Any chance you will fix this bug? :)

@maxi1134
Copy link

This bug is still present. Any update? :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants