[UPDATED] Check Open Windows & Doors

šŸš§ Developers
84

You are welcome!

85

Excellent looking app so far. One suggestion, can you make the trigger accept multiple inputs? because i want it to notify for both heating and cooling. would be good to wrap it up in one instead of two child apps.

1 Like
86

@cuboy29
Nope! Sorry, I can't do that
They are two totally different triggers and what happens if something goes wrong and they both report at the same time?
It wouldn't work unless I completely rewrite the handler method (which is why I did it that way :slight_smile: )

Andy

1 Like
87

enjoy some beers :slight_smile:

1 Like
88

@Christopher @cuboy29

Thanks to both of you for your donations guys!
It's very much appreciated :slight_smile:

Andy

1 Like
89

Guys,
this has now been added to 'Cobra apps'

Andy

1 Like
90

Hi @Cobra,
I'm already using this App to check if all windows/blinds are closed through Alexa, I'd like to move here too some RM rules that I have to send an alert in case something is open and thermostat is heating.
But I have two questions:

  • "message delay" means "alert only if it's open from more than 1 minute" or it's more like "just speech 1 minute after the event in any case, even if it has been closed before 1 minute"?
  • is it possible to have heating setpoint > X degrees as trigger? Actually it's triggered when thermostat is heating, but I'd like to alert also if thermostat is not heating because it has already reached the setpoint (and it could restart heating if door/windows stays open for too much time)
    Thanks
91

Can you tell me if I have this setup correctly? I have it set to run on mode change 'night'. When I arm my house at night it changes to 'armed night'. I don't use 'night' for anything besides arming at bedtime. But this should be the same thing right? If so, it failed for me last night. Left my garage door open on accident and it didn't trigger. Tested tonight by arming and leaving a door open and still not getting any feedback. It's set as a speech synth device with an echo dot

92

@Cobra I just switched to using the cobra container app and updated to version 2.50 of ā€œcheck windows and doors are openā€

I noticed that an additional comma was appearing at the start of the list of contacts (I donā€™t think think this was occurring in the older version I was running)..

image
image1242Ɨ351 52.9 KB

93

There is an extra comma in line 290

state.fullMsg1 = "$newmsg , ${open.join(',')}")

Should be...

state.fullMsg1 = "$newmsg ${open.join(',')}")

Dunno how it managed to get in there :slight_smile:

Andy

94

All fixed - Thanks

1 Like
95

If I use a button push to trigger the app it works as expected. If I use Time it fails with
app:11562019-06-11 11:49:00.111 am errorjava.lang.NullPointerException: Cannot get property 'value' on null object on line 246 (evtHandler)

96

Iā€™ve looked at this and I think Iā€™ve found the error.
But itā€™s almost midnight now so Iā€™ll address it tomorrow.

Andy

97

Bug squashed!

New child updated on GitHub
Version 2.5.1

Andy

1 Like
98

Thank you sir!

1 Like
99

It should be ok now but if you get any problems please let me know.

Andy

100

I had been triggering this with a RM rule. I will modify what I have and check it out.

1 Like
101

@Cobra any chance to have this merged?

I'd avoid to merge each time new changes..

Thanks!

102

Honestly?
Nope..
I don't accept pull/merge requests
I did it once and it caused me lots of problems as I have to support the software I release

I don't even look at these type of GitHub requests any more.

Your code is probably perfect but I haven't looked.

If you just want to trigger on HSM Status then I can probably put that in when I get a chance.
I'll look at you code, and if it is how I would code it, then I'll add it.
If not, then I can soon create a trigger for HSM status

Andy

2 Likes
103

I understand your concerns, basically it just adds HSM Status change as a trigger and I've tried to keep code as much as similar to your.
Feel free to have a look or rewrite it when you have time.