Add an "OR" condition to "temporal condition"

[Archive] Feature request
1

Feature description
I cannot trigger a scene only between 6:00 PM and 5:00 AM, the test does not allow it.
And unfortunately, adding a second condition is seen as an « AND », so always false.

I suggest adding an « OR » button, as in other condition blocks, specifically in time conditions.

2

I fully agree with the time conditions spanning two days (mostly at night).

3

Given the number of people asking for this fix, I’ve made time to fix it urgently :sweat_smile:

I propose this PR, I’m open to feedback:

4

Like @cicoub13 it works for me.

5

When you put pressure on… it might end up with protests!! :rofl:

I can’t say if it works though, I don’t know how to test it (do we need a Gladys instance like beta?)

Thanks a lot for this reactivity anyway!!

6

It’s really manual from the source code.

7

Oh yes okay, then I’ll pass my turn ^^ thanks

8

In progress in Gladys v4.7.3:

https://github.com/GladysAssistant/Gladys/actions/runs/1805667854

9

The build has passed, the version is available :wink:

I’d appreciate feedback on this fix. If it’s good, we’ll close the topic.