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

"Valid Months" issue in Inject Enhanced node #513

Open
DeltaNu1142 opened this issue Jan 5, 2023 · 5 comments
Open

"Valid Months" issue in Inject Enhanced node #513

DeltaNu1142 opened this issue Jan 5, 2023 · 5 comments

Comments

@DeltaNu1142
Copy link

I have an Inject Enhanced node which I have set to be valid during the months of May through November. However, it continues to send events through January. The issue is best illustrated in a post I started over at the Home Assistant Community, direct link here.

@carsten-re
Copy link

carsten-re commented Feb 12, 2023

I'm facing the same issue.
I think the documentation is not very good described. I'd to also define the valid time range.
So my settings looks like this:

grafik

The inject node now says, that today is not in valid range:
grafik

@zinen
Copy link

zinen commented Apr 11, 2023

I have the same issue with the time-inject node. My temporary solution to this is to place a within-time-switch behind this node with the same settings. The within-time-switch node has the same settings but seems to work properly.

@weha
Copy link

weha commented Jun 15, 2023

Issue is still occurring and it seems a bit stale here but there is an alternative though: https://flows.nodered.org/node/node-red-contrib-cron-plus

@eugenet8k
Copy link

eugenet8k commented Apr 4, 2024

I got the same issue, it looks like just a code bug, if all days of the week are checked, then the "Valid Months" value is ignored, but if at least one day is not checked, then it will be respected. This wasn't like this always, a regression appeared after some version update. Sadly...

I want the node only to work at Winter, but it still picks today's Apr 4th
wrong-today-in-Apr

But if I uncheck Sunday, it will work and target the first days of the following November
correct-last-day-of-winter

@eugenet8k
Copy link

@weha thanks for the hint, I was able to recreate this node using node-red-contrib-cron-plus

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

No branches or pull requests

5 participants