-
Notifications
You must be signed in to change notification settings - Fork 6
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
Feature request : Allow limiting by topic #9
Comments
I ended up building my own that does that, just a simple node up on the flow site. https://flows.nodered.org/flow/80390da72f235b760f853a4bda79668b |
have you looked at node-red-contrib-topic-timeframe-trigger |
Ahh, no, hadn't, thanks, looks pretty ideal! This looks pretty good too https://flows.nodered.org/node/node-red-contrib-timeframerlt |
The one you mentioned dies with a weird error I couldn't debug, and doesn't allow passing on the topic and other data (only fixed stuff), so I've updated mine to have a required min count. |
I've got some use cases (say notifications with lights, noise, and phone push messages) where I want to rate limit, but only per topic.
Would you consider adding a "group by" or similar that you can set to topic or any field?
Without this, it's very messy, having to break out in my reusable alert flow to one throttle node per topic, then back into the flow, and maintain all the possible callers.
Cheers
The text was updated successfully, but these errors were encountered: